wangyong9999 commented on PR #314: URL: https://github.com/apache/paimon-cpp/pull/314#issuecomment-5691967347
Thanks @lucasfang and @lxy-9602. Addressed in c7579e7c: the PR now contains only OffsetIndex memoization, direct-plan decision reuse, the focused reuse test and documentation. Page selection is identical to current main; the cursor rewrite and the three extra tests are gone. The doc gives a concrete memory estimate: 1,024 row groups × 10 accessed columns × 1,000 pages × 24 bytes ≈ 234 MiB for parsed locations, plus serialized buffers and container overhead. The count limit is not a byte cap. Updated against main aae05ccb. Local Parquet 226/226, read integration 302/302, read-with-index 68/68, full pre-commit and Apache RAT passed. The PR/issue describe only the final scope and do not claim measured throughput gains for it. Upstream CI is waiting for fork-workflow approval (action_required); the approval API returns HTTP 403 for my account. Could a maintainer approve the workflows for this revision? The identical revision is also running the existing build/sanitizer matrix on the fork: https://github.com/wangyong9999/paimon-cpp/actions/runs/35054485827. -- 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]
