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% make a
difference?
It improves test stability (on highly loaded hosts and in heavy testing
modes), but I doubt it's the right way to go. I'd measure actual startup
time and adjust timeout value, but it seems jtreg doesn't provide such
info.
So, I'd leave it as is for now, file an RFE on jtreg and rewrite this
logic once support in jtreg is implemented.
Otherwise, looks good.
Best regards,
Vladimir Ivanov
Paul.
On Nov 29, 2014, at 5:36 PM, Igor Ignatyev <igor.ignat...@oracle.com> 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 if it's not enough time to continue
bug : https://bugs.openjdk.java.net/browse/JDK-8039953
changes in testlibrary : https://bugs.openjdk.java.net/browse/JDK-8066191
testing: locally
--
Igor
