asafm commented on issue #13869: URL: https://github.com/apache/pulsar/issues/13869#issuecomment-1283665018
Hi @ofek, you are absolutely correct. I've been working over the last several months documenting the current state of metrics and releasing the document to the community just 2-3 weeks ago. As you can see in [there](https://docs.google.com/document/d/1vke4w1nt7EEgOvEerPEUS-Al3aqLTm9cl2wTBkKNXUA/edit#heading=h.9m4eafn77fuo), it's a known issue. This document is part of a large effort to refactor how the metrics are defined, used, and exported in Pulsar. @codelipenghui @merlimat - we potentially don't have to wait for the full refactor, but provide a fix just for exporting histograms - it's not a small fix, but it's not a complicated fix. the *biggest* issue is once we do that of course, we break compatibility, so this must be done gradually with flags (`oldHistogram=true, newHistogram=false`). WDYT? -- 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]
