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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to