Skip to content

Conversation

lampnick
Copy link
Contributor

What is the purpose of the change

fix: wrong stats log(#907 )

Brief changelog

fix: wrong stats log(#907 )

Verifying this change

fix: wrong stats log(#907 )

@codecov-commenter
Copy link

codecov-commenter commented Aug 25, 2022

Codecov Report

Merging #908 (1a0fc5b) into master (2630383) will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##           master     #908   +/-   ##
=======================================
  Coverage   23.65%   23.65%           
=======================================
  Files          52       52           
  Lines        7150     7150           
=======================================
  Hits         1691     1691           
  Misses       5229     5229           
  Partials      230      230           
Impacted Files Coverage Δ
consumer/statistics.go 72.85% <33.33%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@vongosling vongosling changed the title fix:wrong stats log (#907) [ISSUE #907]remove useless stats log (#907) Aug 26, 2022
@vongosling
Copy link
Member

would you like to add a new logic in another pr, to show sum and qps?

@vongosling vongosling linked an issue Aug 26, 2022 that may be closed by this pull request
@vongosling vongosling merged commit 398ae67 into apache:master Aug 26, 2022
@lampnick
Copy link
Contributor Author

would you like to add a new logic in another pr, to show sum and qps?

The sum and tps was in current code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stats log maybe wrong
4 participants