Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

collect publish rate #285

Merged
merged 2 commits into from
Dec 28, 2020
Merged

Conversation

dockerzhang
Copy link
Contributor

fixes #284

pulsar will add a updateRates method for this PR.

@dockerzhang
Copy link
Contributor Author

need apache/pulsar#9049 to be merged first.

@dockerzhang
Copy link
Contributor Author

@BewareMyPower apache/pulsar#9049 was merged, please update the dependency.

@BewareMyPower
Copy link
Collaborator

OK, we'll first wait for new sn-pulsar released. Then I'll provide a pulsar version and you can just update the dependency in your PR.

@BewareMyPower
Copy link
Collaborator

2.8.0-rc-202012272229

Copy link
Collaborator

@BewareMyPower BewareMyPower left a comment

Choose a reason for hiding this comment

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

LGTM. Good job!

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

Successfully merging this pull request may close these issues.

[BUG] msgRateIn and msgThroughputIn is 0
2 participants