Hi Igor,
This looks ok.
I like how you have factored out things into TimeLimitedRunner. Do you plan in
a future patch to update lambda form test code that uses similar functionality?
Is the adjustment timeout *= 0.9 necessary? does reduction by 10% make a
difference?
Paul.
On Nov 29, 2014,
On 12/1/14, 2:46 PM, Paul Sandoz wrote:
Hi Igor,
This looks ok.
I like how you have factored out things into TimeLimitedRunner. Do you plan in
a future patch to update lambda form test code that uses similar functionality?
Is the adjustment timeout *= 0.9 necessary? does reduction by 10%
http://cr.openjdk.java.net/~iignatyev/8039953/webrev.01/
to have TimeLimitedRunner more general, I've added 'factor' argument to
its ctor.
Thanks,
Igor
On 12/01/2014 02:39 PM, Igor Ignatyev wrote:
Paul/Vladimir,
thanks for review.
On 12/01/2014 01:29 PM, Vladimir Ivanov wrote:
On
Looks good.
Best regards,
Vladimir Ivanov
On 12/1/14, 10:58 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev/8039953/webrev.01/
to have TimeLimitedRunner more general, I've added 'factor' argument to
its ctor.
Thanks,
Igor
On 12/01/2014 02:39 PM, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev/8039953/webrev.00
98 lines changed: 93 ins; 3 del; 2 mod;
Hi all,
Please review the patch:
Problem:
on some configurations,
java/lang/invoke/MethodHandles/CatchExceptionTest.java can timeout
before all test cases are run
Fix:
interrupt test execution
