Re: [PR] [core] Support push-down valueStats prune when read manifests. [paimon]

["baiyangtx (via GitHub)" <gi...@apache.org>]

baiyangtx commented on PR #7392:
URL: https://github.com/apache/paimon/pull/7392#issuecomment-4553338588

   
   Thanks for the review, @JingsongLi 
   
   **Cache key:** The cache stores raw `InternalRow` bytes, not deserialized objects. Deserialization, filtering, and conversion run fresh on every cache hit. `dropStats` is applied as a post-processing loop after the cache read returns, so the same cached bytes are safe for both `dropStats=true` and `dropStats=false` queries.
   
   **Correctness:** `filterByStats(entry)` executes inside `ManifestFile.read()` before the post-processing loop, so it always sees complete stats. And `dropStats` defaults to `false` — unaffected paths remain untouched.
   
   Fixed the PR title typo as well.


-- 
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: issues-unsubscribe@paimon.apache.org

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