Code Review - CR #7114195 -Short timeouts on regression tests (original) (raw)
Alan Bateman Alan.Bateman at oracle.com
Fri Dec 23 09:51:33 UTC 2011
- Previous message: Code Review - CR #7114195 -Short timeouts on regression tests
- Next message: Request for review: 7123582: (launcher) display the -version and -XshowSettings
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 22/12/2011 15:08, Gary Adams wrote:
:
Webrev at: http://cr.openjdk.java.net/~gadams/7114195/ Dropping the /timeout from tests that specify <= 120s should be fine (and skimming through the patch, I don't see any issues). All it really means is that if the test hangs or deadlocks then it won't failed until the default timeout elapses. Many of these short timeouts were likely included when the test was originally created to make it quicker to test when running on a JDK without the corresponding fix.
-Alan
- Previous message: Code Review - CR #7114195 -Short timeouts on regression tests
- Next message: Request for review: 7123582: (launcher) display the -version and -XshowSettings
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]