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

Using `volatile + ConcurrentHashMap + LongAdder` instead of `syncrhonzied + HashMap + AtomicLong` for CounterGroup #194

Open
wants to merge 1 commit into
base: trunk
from

Conversation

Projects
None yet
2 participants
@asdf2014
Member

asdf2014 commented Jan 17, 2018

Using volatile + ConcurrentHashMap + LongAdder instead of syncrhonzied + HashMap + AtomicLong for CounterGroup.

Get more details from my blog: yuzhouwan.com.

@asfgit asfgit force-pushed the apache:trunk branch from ffc5554 to 286c244 Feb 5, 2018

Using `volatile + ConcurrentHashMap + LongAdder` instead of `syncrhon…
…zied + HashMap + AtomicLong` for CounterGroup
@asdf2014

This comment has been minimized.

Member

asdf2014 commented Feb 24, 2018

Already rebase those commits on trunk branch, and travis ci system have passed. BTW, it seems that -DskipTests param makes all test cases skipped. @szaboferee

@asfgit

This comment has been minimized.

asfgit commented Aug 17, 2018

Can one of the admins verify this patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment