liujiayi771 opened a new pull request, #190: URL: https://github.com/apache/paimon-cpp/pull/190
### Purpose Linked issue: N/A This is a follow-up to #187 and adds full support for `manifest.delete-file-drop-stats=true` in the C++ commit path. - Parse and expose the existing Paimon table option. - Drop `value_stats` and `value_stats_cols` from final DELETE manifest entries. - Apply the optimization after scan filtering and pruning for overwrite and append compaction paths. - Preserve complete statistics for ADD entries, generic metadata scans, and writer restore. - Keep writer restore statistics until the final entry kind is known, intentionally avoiding the metadata-only level-upgrade bug tracked in https://github.com/apache/paimon/issues/7026. ### Tests - `cmake --build build --target paimon_static paimon_shared -j8` - `cmake --build build --target paimon-core-test -j8` - `./build/debug/paimon-core-test --gtest_brief=1` (1629 tests passed) - `pre-commit run --files <changed files>` - `git diff --check apache/main...HEAD` ### API and Format Adds the public option key `kManifestDeleteFileDropStats`. There is no manifest schema or protocol change. When enabled, DELETE entries store empty value statistics and an empty value-statistics column list, matching the existing Paimon format. ADD entries retain complete statistics for file skipping. ### Documentation This implements an existing Paimon table option. No documentation files are included in this PR. ### Generative AI tooling Generated-by: Codex (GPT-5) and 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]
