Re: 8141370: com/sun/jndi/ldap/LdapTimeoutTest.java failed intermittently

2015-12-09 Thread Vincent Ryan
This change to restructure the test looks fine. Thanks. > On 8 Dec 2015, at 16:11, Rob McKenna wrote: > > Thanks, just spotted that myself after attempting to run it in a loop via > jtreg. (in case jtreg is somehow involved in causing the intermittent > failure) Will be corrected pre-push. >

Re: 8141370: com/sun/jndi/ldap/LdapTimeoutTest.java failed intermittently

2015-12-08 Thread Rob McKenna
Thanks, just spotted that myself after attempting to run it in a loop via jtreg. (in case jtreg is somehow involved in causing the intermittent failure) Will be corrected pre-push. -Rob On 08/12/15 15:45, Daniel Fuchs wrote: Hey Rob, The @run line seems to be wrong in the new test fi

Re: 8141370: com/sun/jndi/ldap/LdapTimeoutTest.java failed intermittently

2015-12-08 Thread Daniel Fuchs
Hey Rob, The @run line seems to be wrong in the new test file: 26 * @run main/othervm LdapTimeoutTest cheers, -- daniel On 08/12/15 15:00, Rob McKenna wrote: Hi folks, Hopefully this fix puts this tests' failures to bed. (mind you I've thought that before) Basically I've separated the fa