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
Use atomic field updaters in selected Uni places #669
Conversation
jponge
commented
Aug 31, 2021
- Avoid allocating excessive amounts of AtomicReference instances
- Improve abstraction over internal state of UniOperatorProcessor
- Use further atomic field updaters in selected places (Uni)
- Use CaS methods in UniOperatorProcessor
@Sanne You can review this |
Codecov Report
@@ Coverage Diff @@
## main #669 +/- ##
============================================
+ Coverage 89.85% 89.98% +0.12%
- Complexity 2976 2981 +5
============================================
Files 374 374
Lines 11752 11757 +5
Branches 1476 1476
============================================
+ Hits 10560 10579 +19
+ Misses 611 604 -7
+ Partials 581 574 -7
|
Supersedes #660 |
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!
I'll be testing it soon with the Techempower benchmark as well, but I see no reason to wait for it. |