visortelle commented on code in PR #21635:
URL: https://github.com/apache/pulsar/pull/21635#discussion_r1417609104


##########
pip/pip-320.md:
##########
@@ -0,0 +1,241 @@
+# PIP-320 OpenTelemetry Scaffolding 
+
+# Background knowledge
+
+## PIP-264 - parent PIP titled "Enhanced OTel-based metric system"
+[PIP-264](https://github.com/apache/pulsar/pull/21080), which can also be 
viewed [here](pip-264.md), describes in high 
+level a plan to greatly enhance Pulsar metric system by replacing it with 
[OpenTelemetry](https://opentelemetry.io/).
+You can read in the PIP the numerous existing problems PIP-264 solves. Among 
them are:
+- Control which metrics to export per topic/group/namespace via the 
introduction of a metric filter configuration
+- Reduce the immense metrics cardinality due to high topic count (One of 
Pulsar great features), by introducing

Review Comment:
   It would be useful to have some best practices/recipes/recommendations in 
the documentation on which cases I should use groups, and in which not. With 
the attached benchmark results.
   In other words, it would be nice to have some kind of algorithm for when and 
how to group metrics.
   
   Maybe not related to the PIP itself.



-- 
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-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to