New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bulkhead TCK tests - timing issue in TestData.check() #227
Milestone
Comments
mkouba
added a commit
to mkouba/wildfly-swarm
that referenced
this issue
Mar 5, 2018
…Failures() - this test has a timing issue and fails intermittently - see also eclipse/microprofile-fault-tolerance#227
mkouba
added a commit
to mkouba/wildfly-swarm
that referenced
this issue
Mar 5, 2018
…ToClassRetryFailures() - this test has a timing issue and fails intermittently - see also eclipse/microprofile-fault-tolerance#227
kenfinnigan
pushed a commit
to thorntail/thorntail
that referenced
this issue
Mar 5, 2018
…ToClassRetryFailures() (#821) - this test has a timing issue and fails intermittently - see also eclipse/microprofile-fault-tolerance#227
@hutchig can you take a look at this one? |
@Emily-Jiang OK. Will do. |
I have moved the latch decrement to the finally block after the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The number of workers may be decremented AFTER the count of the latch is decremented. See also https://github.com/eclipse/microprofile-fault-tolerance/blob/master/tck/src/main/java/org/eclipse/microprofile/fault/tolerance/tck/bulkhead/clientserver/Checker.java#L111
As a result, an assert in TestData.check() may fail incorrectly.
We've managed to reproduce the issue in WildFly Swarm with
org.eclipse.microprofile.fault.tolerance.tck.bulkhead.BulkheadAsynchRetryTest.testBulkheadQueReplacesDueToClassRetryFailures()
.The text was updated successfully, but these errors were encountered: