Re: RFR 8072030 Race condition in ThenComposeExceptionTest.java

2015-02-03 Thread Chris Hegarty
On 2 Feb 2015, at 11:36, Paul Sandoz paul.san...@oracle.com wrote: ... This was a nasty race. In fact there is no guarantee that the CF, that sets the AtomicReference would even complete, without join/get, right ? I think the async completion of the thenComposed task will trigger (in the

Re: RFR 8072030 Race condition in ThenComposeExceptionTest.java

2015-02-02 Thread Chris Hegarty
On 02/02/15 09:29, Paul Sandoz wrote: Hi, http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8072030-thenComposeException-test-race-condition/webrev/ I introduced a silly race condition in the test ThenComposeExceptionTest.java. This manifested itself in hotspot testing where various VM options

Re: RFR 8072030 Race condition in ThenComposeExceptionTest.java

2015-02-02 Thread Paul Sandoz
On Feb 2, 2015, at 11:41 AM, Chris Hegarty chris.hega...@oracle.com wrote: On 02/02/15 09:29, Paul Sandoz wrote: Hi, http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8072030-thenComposeException-test-race-condition/webrev/ I introduced a silly race condition in the test

RFR 8072030 Race condition in ThenComposeExceptionTest.java

2015-02-02 Thread Paul Sandoz
Hi, http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8072030-thenComposeException-test-race-condition/webrev/ I introduced a silly race condition in the test ThenComposeExceptionTest.java. This manifested itself in hotspot testing where various VM options increased the probability of failure.