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

Expose pulsar_out_bytes_total and pulsar_out_messages_total for namespace/subscription/consumer. #6918

Merged
merged 2 commits into from
May 8, 2020

Conversation

codelipenghui
Copy link
Contributor

@codelipenghui codelipenghui commented May 8, 2020

Fixes #6891
Rated to #5802

Motivation

Add pulsar_out_bytes_total and pulsar_out_messages_total for namespace/subscription/consumer.

Verifying this change

New unit test added.

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@sijie sijie merged commit 204f327 into apache:master May 8, 2020
@codelipenghui codelipenghui deleted the in_out_count branch May 9, 2020 01:37
jiazhai pushed a commit that referenced this pull request May 12, 2020
…pace/subscription/consumer. (#6918)

Fixes #6891
Rated to #5802

Add pulsar_out_bytes_total and pulsar_out_messages_total for namespace/subscription/consumer.

New unit test added.
(cherry picked from commit 204f327)
cdbartholomew pushed a commit to kafkaesque-io/pulsar that referenced this pull request May 23, 2020
…pace/subscription/consumer. (apache#6918)

Fixes apache#6891
Rated to apache#5802

Add pulsar_out_bytes_total and pulsar_out_messages_total for namespace/subscription/consumer.

New unit test added.
Huanli-Meng pushed a commit to Huanli-Meng/pulsar that referenced this pull request May 27, 2020
…pace/subscription/consumer. (apache#6918)

Fixes apache#6891
Rated to apache#5802
### Motivation

Add pulsar_out_bytes_total and pulsar_out_messages_total for namespace/subscription/consumer.

### Verifying this change

New unit test added.
addisonj pushed a commit to instructure/pulsar that referenced this pull request Jun 12, 2020
…pace/subscription/consumer. (apache#6918)

Fixes apache#6891
Rated to apache#5802

Add pulsar_out_bytes_total and pulsar_out_messages_total for namespace/subscription/consumer.

New unit test added.
(cherry picked from commit 204f327)
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
…pace/subscription/consumer. (apache#6918)

Fixes apache#6891
Rated to apache#5802
### Motivation

Add pulsar_out_bytes_total and pulsar_out_messages_total for namespace/subscription/consumer.

### Verifying this change

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

Successfully merging this pull request may close these issues.

Expose topic/subscription/consumer number of messages as counter.
2 participants