wirybeaver opened a new pull request, #920: URL: https://github.com/apache/iceberg-cpp/pull/920
## Summary - add the general `position_deletes` metadata-table kind and materialized scan API - expand live Parquet position-delete files and v3 Puffin deletion vectors into logical rows - support partitioned and unpartitioned tables, upgraded mixed tables, multi-blob Puffin files, and projected/reordered output schemas - expose v3 `content_offset` and `content_size_in_bytes` metadata Refs #916. Depends on #913 and #914. This branch includes those prerequisite commits. ## Test plan - `cmake --build build-clang --target metadata_table_test position_deletes_table_test -j2` - `metadata_table_test` — 4 passed - `position_deletes_table_test` — 6 passed - `data_test --gtest_filter='DVWriterTest.*:DeleteLoaderTest.*'` — 30 passed - `puffin_test --gtest_filter='DVUtilTest.*:PuffinDVInteropTest.*'` — 8 passed - `util_test --gtest_filter='PositionDeleteIndexTest.*'` — 23 passed - `uvx pre-commit run --all-files` — passed ## Known limitation The optional deleted `row` column is currently emitted as null. The existing reusable C++ position-delete reader/writer path only exposes the required `file_path` and `pos` columns; preserving externally written row-bearing deletes requires a follow-up reader abstraction. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
