wangyong9999 commented on PR #267: URL: https://github.com/apache/paimon-cpp/pull/267#issuecomment-5491954741
Agreed. I removed the process-wide reader and BTree cache registry, namespace, and allocator-lifetime changes. The query engine now owns cross-query `GlobalIndexReader` reuse through the existing public API, pins the data scan to the same snapshot, and honors `IsThreadSafe()` around shared-reader evaluation. This PR is now limited to reusing the schema, snapshot context, executor, memory pool, and caller-provided manifest cache already available inside one `TableScan`. The resulting diff is 4 files; `paimon-global-index-test` passes 132/132. -- 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]
