Skip to content
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

[ISSUE #110] metrics bugfix: switch getTps() to getSum() #108

Merged
merged 1 commit into from
Aug 25, 2022

Conversation

carlvine500
Copy link

@carlvine500 carlvine500 commented Jun 30, 2022

What is the purpose of the change

close #110
several metrics not correct

Brief changelog

metrics bugfix: switch getTps() to getSum()

Copy link
Member

@ShannonDing ShannonDing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ShannonDing ShannonDing added the bug Something isn't working label Jul 31, 2022
@ShannonDing ShannonDing requested a review from hdchen July 31, 2022 08:18
@ShannonDing ShannonDing changed the title metrics bugfix: switch getTps() to getSum() [ISSUE #110] metrics bugfix: switch getTps() to getSum() Aug 25, 2022
@ShannonDing ShannonDing merged commit 290bcdb into apache:master Aug 25, 2022
@sinrimin sinrimin mentioned this pull request Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sum value of topic metrix error in broker stats.
2 participants