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.

Sign up for GitHub

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

Jump to bottom

[improve][client] Print consumer stats log if prefetched messages are not zero #23698

Merged
merged 1 commit into from
Dec 12, 2024
Merged

[improve][client] Print consumer stats log if prefetched messages are not zero #23698

merged 1 commit into from
Dec 12, 2024
+3 −2

Conversation

codelipenghui
Copy link
Contributor

Motivation

Print consumer stats log if prefetched messages are not zero to help users know that the consumer just stop consuming messages from the receiver queue. Now, we only print the stats log if there is receive or ack throughput on the consumer.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@codelipenghui codelipenghui self-assigned this Dec 9, 2024
@codelipenghui codelipenghui added type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages area/client release/4.0.2 labels Dec 9, 2024
@codelipenghui codelipenghui added this to the 4.1.0 milestone Dec 9, 2024
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Dec 9, 2024
@dao-jun dao-jun merged commit c52af1d into apache:master Dec 12, 2024
58 of 62 checks passed
lhotari pushed a commit that referenced this pull request Dec 16, 2024
@codelipenghui @lhotari
[improve][client] Print consumer stats log if prefetched messages are…
8f9d796 
… not zero (#23698)

(cherry picked from commit c52af1d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dao-jun dao-jun dao-jun approved these changes

@pdolif pdolif pdolif approved these changes

Assignees

@codelipenghui codelipenghui

Labels
area/client cherry-picked/branch-4.0 doc-not-needed Your PR changes do not impact docs ready-to-test release/4.0.2 type/enhancement The enhancements for the existing features or docs. e.g. reduce memory usage of the delayed messages
Projects
None yet
Milestone
4.1.0
Development

Successfully merging this pull request may close these issues.
4 participants
@codelipenghui @dao-jun @pdolif @lhotari