-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
HDDS-1870. ConcurrentModification at PrometheusMetricsSink #1179
Conversation
/label ozone |
💔 -1 overall
This message was automatically generated. |
@elek please review |
LGTM +1 (non binding) |
💔 -1 overall
This message was automatically generated. |
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.
+1 LGTM.
Thank You @adoroszlai for the contribution. |
Thank you @bharatviswa504 for reviewing and merging it. |
What changes were proposed in this pull request?
ConcurrentModification
due to plainHashMap
https://issues.apache.org/jira/browse/HDDS-1870
How was this patch tested?
Ran freon in
ozoneperf
env. Checked/prom
endpoints. Verified that no CME appears in logs.