-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Introducing JCStress tests for concurrent queues #3069
Conversation
@chemicL Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
@chemicL Thank you for signing the Contributor License Agreement! |
4f82e31
to
7e283d8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me, with a nitpick on formatting (rest of the code, or at least the recent parts, use tabs instead of spaces)
reactor-core/src/jcstress/java/reactor/util/concurrent/SpscArrayQueueStressTest.java
Outdated
Show resolved
Hide resolved
7e283d8
to
d4a9733
Compare
Thanks. I applied the idea code_style.xml and improved the formatting. |
Concurrent queues from `reactor.util.concurrent` package are now exercised against the contractual obligations in their respective scenarios (multiple-producer-single-consumer and single-producer-single-consumer).
d4a9733
to
337c356
Compare
@simonbasle this PR seems to have been merged on a maintenance branch, please ensure the change is merge-forwarded to intermediate maintenance branches and up to |
Concurrent queues from
reactor.util.concurrent
package are nowexercised against the contractual obligations in their respective
scenarios (multiple-producer-single-consumer and
single-producer-single-consumer).