coderzc opened a new pull request, #25848: URL: https://github.com/apache/pulsar/pull/25848
### Motivation Ledger offloader implementations can keep their own read caches, but Pulsar currently does not expose a broker-owned metric path for cache hit/miss analysis. This makes it hard to diagnose tiered-storage read performance and cache sizing issues through the broker OpenTelemetry pipeline. ### Modifications - Add `recordReadOffloadCacheHit` and `recordReadOffloadCacheMiss` to `LedgerOffloaderStats`. - Register broker OpenTelemetry counters for ledger offloader cache operation count and size. - Emit attributes for namespace, optional topic, and hit/miss status using the same topic-level setting as existing offloader stats. - Pass broker OpenTelemetry into `LedgerOffloaderStats` from `PulsarService`. - Add broker test coverage for the new OpenTelemetry metrics. ### Verifying this change - `./gradlew :pulsar-broker:test --tests org.apache.pulsar.broker.stats.LedgerOffloaderMetricsTest.testReadOffloadCacheOpenTelemetryMetrics :managed-ledger:checkstyleMain :pulsar-opentelemetry:checkstyleMain :pulsar-broker:checkstyleMain :pulsar-broker:checkstyleTest` -- 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]
