RFR: 8236108 tools/javac/lambda/LambdaParserTest.java timed out

[Adam Sotona]

Thanks for the review,
Adam

> On 8 Jun 2020, at 22:34, Vicente Romero <vicente.romero at oracle.com> wrote:
> 
> Hi Adam,
> 
> webrev.01 looks good to me,
> 
> Thanks,
> Vicente
> 
> On 6/2/20 9:41 AM, Adam Sotona wrote:
>> Hi,
>> I would like to ask for review of LambdaParserTest.java timed out fix.
>> This particular test is very complex and intensive.
>> It includes 573696 test actions, each parsing a source fragment.
>> As this single test takes on my powerful machine approx. 1 minute and the default timeout is 2 minutes, so I can imagine situations where it timeouts.
>> 
>> Target machine scale factor was 4, so the default timeout was 480s, however the test elapsed time was 523s.
>> 
>> Proposed patch extends the timeout by 50% to 180s, so multiplied by the machine-specific timeout scale factor it will give enough time to finish successfully.
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8236108
>> webrev: http://cr.openjdk.java.net/~asotona/8236108/webrev.00/
>> 
>> The test is a part of Tier 4 and it is passing for me even without the extended timeout, so I cannot practically verify the patch effectivity unless it fails.
>> 
>> Thanks,
>> Adam
>