[8u] RFR: 8249677: Regression after JDK-8237117: Better ForkJoinPool behavior

[Anton Kozlov]

Hi,

On 23.07.2020 17:54, Severin Gehwolf wrote:
> On Thu, 2020-07-23 at 11:46 -0300, Martin Balao wrote:
>>
>> Even though your Webrev.03 patch has been reviewed and
>> maintainer-approved, I believe we should revisit the 'testDefault'
>> decision before pushing. It's not clear to me yet whether 'testDefault'
>> behaviour will be 'expected' or 'unexpected'. In any case, that will be
>> addressed in the context of 8249846 [1].
> 
> Yes, agreed.
> 
> To proceed with this fix I suggest the following: Please remove
> testDefault code and references to 8249846
> from test/java/util/concurrent/forkjoin/AccessControlContext.java
> It wouldn't be the right call to commit that under 8249677. The test
> and the fix (if any) should come with the changeset for 8249846.

I personally don't have a strict opinion about the test case.
I specifies the behavior of 8u262, I don't think it is certainly 
wrong to have one before 8249846.

For simplicity of logistics, we can remove the case until resolution 
for 8249846. Not sure what we'll decide for the test case if we also 
decide not to fix 8249846. But this is not really important now, I agree.
New webrev with proposed changes:

  http://cr.openjdk.java.net/~akozlov/8249677/webrev.04/

I would like Andrew to acknowledge he have seen the new revision, to avoid
missing expectations.

Thanks,
Anton