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

[Doc] The description of messagesConsumedCounter is wrong #23702

Open
2 tasks done
BewareMyPower opened this issue Dec 10, 2024 · 1 comment
Open
2 tasks done

[Doc] The description of messagesConsumedCounter is wrong #23702

BewareMyPower opened this issue Dec 10, 2024 · 1 comment
Labels
doc-required Your PR changes impact docs and you will update later.

Comments

@BewareMyPower
Copy link
Contributor

Search before asking

  • I searched in the issues and found nothing similar.

What issue do you find in Pulsar docs?

See https://pulsar.apache.org/docs/next/administration-stats/#topic-internal-stats

messagesConsumedCounter The number of messages this cursor has acknowledged since this broker loads this topic.

Actually the description is wrong. It's the number of entries, not messages.

What is your suggestion?

Fix the description

Any reference?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@BewareMyPower BewareMyPower added the doc-required Your PR changes impact docs and you will update later. label Dec 10, 2024
@lhotari
Copy link
Member

lhotari commented Dec 10, 2024

Mixing message counts and entries counts is also elsewhere in Pulsar, example #23263

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
doc-required Your PR changes impact docs and you will update later.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lhotari @BewareMyPower