strimzi / metrics-reporter

Prometheus Metrics Reporter for Apache Kafka server and client compoenents
Apache License 2.0
4 stars 7 forks source link

Potential performance improvements in KafkaPrometheusMetricsReporter #9

Closed mimaison closed 3 weeks ago

mimaison commented 7 months ago

We could apply the filtering when adding/removing metrics in KafkaPrometheusMetricsReporter instead of doing it in KafkaMetricsCollector.collect() which runs when the /metrics endpoint is hit so it's on the hot path.

mimaison commented 3 weeks ago

Done in #38