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
fix duplicate typeline for prometheus type #3137
Conversation
@eolivelli @dlg99 PTAL |
@@ -172,6 +172,7 @@ public StatsLogger getStatsLogger(String scope) { | |||
|
|||
@Override | |||
public void writeAllMetrics(Writer writer) throws IOException { | |||
PrometheusTextFormatUtil.reset(); |
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.
This is a sign of something going wrong.
What happens with two concurrent calls to this method?
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.
@eolivelli you are right, we don't know what will happen. Let me push a fix.
1e85564
to
fdea75b
Compare
@eolivelli PTAL again thanks, I refractor For an alternate approach, I can add HashSet params to every method in |
fdea75b
to
729ad13
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.
Lgtm
(cherry picked from commit 38dc328)
(cherry picked from commit 38dc328)
This has already been released since 4.15.0. Remove the 4.15.x label. |
Fixes #3136
Descriptions of the changes in this PR:
Motivation
fix duplicate typeline for prometheus type
Changes
remember the prometheus type, record in set, incase print again
Master Issue: #3136