You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/05/05 02:22:22 UTC
[GitHub] [pulsar] MarvinCai opened a new issue, #15439: [Doc] inappropriate/Outdated description for metric pulsar_storage_backlog_size for namespace
MarvinCai opened a new issue, #15439:
URL: https://github.com/apache/pulsar/issues/15439
### What issue do you find in Pulsar docs?
Description of pulsar_storage_backlog_size for namespace (https://pulsar.apache.org/docs/en/reference-metrics/#namespace-metrics) said this metric is message count.
But looking into the code of how this metric get calculated, it's actually in byte.
### What is your suggestion?
Update to accurate description (in byte) if that's the case.
### Do you have any references?
Code reference:
https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/stats/prometheus/NamespaceStatsAggregator.java#L339
https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/stats/prometheus/NamespaceStatsAggregator.java#L79
https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/stats/prometheus/NamespaceStatsAggregator.java#L157
https://github.com/apache/pulsar/blob/master/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedger.java#L397-L402
### Would you like to fix this issue?
Yes
### Note
- [X] I have researched my question.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet closed issue #15439: [Doc] inappropriate/Outdated description for metric pulsar_storage_backlog_size for namespace
Posted by GitBox <gi...@apache.org>.
Anonymitaet closed issue #15439: [Doc] inappropriate/Outdated description for metric pulsar_storage_backlog_size for namespace
URL: https://github.com/apache/pulsar/issues/15439
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet commented on issue #15439: [Doc] inappropriate/Outdated description for metric pulsar_storage_backlog_size for namespace
Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on issue #15439:
URL: https://github.com/apache/pulsar/issues/15439#issuecomment-1119137192
Hi @momo-jun can you take a look at this issue? Thanks
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] momo-jun commented on issue #15439: [Doc] inappropriate/Outdated description for metric pulsar_storage_backlog_size for namespace
Posted by GitBox <gi...@apache.org>.
momo-jun commented on issue #15439:
URL: https://github.com/apache/pulsar/issues/15439#issuecomment-1119217095
@MarvinCai thanks for filing this issue.
@tjiuming will follow up with the doc updates. Thanks for your volunteering.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org