u70b3 opened a new pull request, #841: URL: https://github.com/apache/iceberg-cpp/pull/841
## Summary - parse the optional task-level `data-sequence-number` in REST file scan tasks - serialize the field when a scan task's data file has a sequence number - enable the row-lineage sequence test and cover absent/null compatibility ## Motivation REST-planned scans can carry `first-row-id`, but C++ currently cannot receive the manifest-entry data sequence number needed to inherit `_last_updated_sequence_number` for v3 tables. Closes #834 ## Compatibility and dependency This is the C++ side only. The task-level `data-sequence-number` field is not yet present in the upstream REST OpenAPI, so this PR remains draft pending agreement and publication of that protocol field. Older servers that omit the field, and explicit `null`, continue to produce an unset data sequence number. ## Testing - `pre-commit run --all-files` - full CMake Debug build with GCC 14 and the repository-pinned CRoaring 4.4.3 - full CTest suite (18/18 passed) - `rest_catalog_test` (317/317 passed) ## AI assistance This change was implemented with assistance from OpenAI Codex. I reviewed the implementation and tests, verified the JSON compatibility behavior, and ran the full lint/build/test checks above. The remaining uncertainty is the final upstream REST OpenAPI field definition; this draft intentionally does not modify the Java SDK or specification. -- 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]
