wangyong9999 opened a new pull request, #303: URL: https://github.com/apache/paimon-cpp/pull/303
### Purpose No linked issue. PK index planning first creates the data plan, whose starting scanner loads the selected snapshot. It then loads the same snapshot again before scanning index metadata. On remote storage this adds another open/read round trip, including when no matching index entries exist. Reuse the snapshot already held by the scan when its ID matches the data plan. Otherwise retain the existing load path. This is per-scan reuse, not a shared cache; snapshot discovery and freshness are unchanged. ### Tests - Extend the historical-snapshot integration test to count snapshot opens while preserving result checks for snapshots 2, 3 and 5. - Local targeted executable: all 12 Parquet/ORC PK sorted-index integration cases pass. Built the changed translation unit and integration test against the matching main-based dependency bundle; this is not a full CMake build. - Negative control using the unmodified implementation fails both format cases: two snapshot opens instead of one. - `pre-commit run --all-files` and `git diff --check` pass. Full CI pending. - Local `-Wall` build has signed-comparison warnings from the older bundled GoogleTest headers; no production-source warning observed. ### API and Format Internal accessors only. No change under `include/`, storage format, or protocol. ### Documentation No new configuration or user-facing feature. ### Generative AI tooling Generated-by: OpenAI Codex (GPT-5) -- 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]
