On 1/3/19 11:37 AM, Leonid Mesnik wrote:

On Jan 3, 2019, at 10:26 AM, Jini George <jini.geo...@oracle.com> wrote:

Thank you very much, Chris, for taking a look. I will modify the timeout value to be 
480 ==> 8 minutes ==> 32 minutes for tier1. Sounds good ?

I think it makes a sense to move this test out of tier1. It takes too long time 
for tier1.
Agreed. I think changing the timeout to 480 is fine, but this test does run too long for tier1.

Chris

Leonid
Thanks!
Jini.

On 1/3/2019 9:54 PM, Chris Plummer wrote:
Hi Jini,
2400 seems excessive. That's 40 minutes, which translates to 160 minutes for 
our test runs. Did you mean timeout=240? It looks like a few of the Clhsdb 
tests were given large timeouts that are probably much more than is needed.
thanks,
Chris
On 1/2/19 6:51 PM, Jini George wrote:
Hello!

Requesting reviews for a small fix for fixing the timeout failures of the test: 
ClhsdbInspect.java by increasing the timeout value.

BugID: https://bugs.openjdk.java.net/browse/JDK-8213457
Webrev: http://cr.openjdk.java.net/~jgeorge/8213457/webrev.00/index.html

The fix has been tested with about 200 repeated runs on windows post this fix. 
The timeout was not observed.

Thanks,
Jini.




Reply via email to