Jackie-Jiang opened a new pull request, #19756: URL: https://github.com/apache/pinot/pull/19756
## Summary Segments backed by a remote storage tier read data remotely whenever it is not in the local cache. This adds two query stats for those reads, `numRemoteAccesses` and `remoteAccessBytes`: - Single-stage engine: new fields on `BrokerResponse`, also copied into `CursorResponseNative`. - Multi-stage engine: new `LeafOperator` stat keys, shown in the leaf `StageStats` and merged into the broker response. Both stay `0` unless a `SegmentDirectory` implementation backed by remote storage records them, which it does through the new `FetchContext#addRemoteAccesses(long, long)` accumulator, typically when the fetched buffers are released. `InstanceResponseOperator` sums the fetch contexts of the request into two new `DataTable` metadata keys (ids 45 and 46), and the broker reduce path, the leaf operator and the time series engine carry them through. ## Compatibility - Older brokers ignore the new metadata keys, and both `StatKey` enums are appended last so the `StatMap` wire ordinals of existing keys do not move. - The new response fields serialize as `0` for deployments without a remote storage tier. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
