The GitHub Actions job "pre-commit" on 
iceberg-cpp.git/implement-data-file-writer has failed.
Run started by GitHub user shangxinli (triggered by shangxinli).

Head commit for run:
90d324ea5d17c09ff5514d3f8efa8215cecec45f / shangxinli 
<[email protected]>
feat: implement DataWriter for Iceberg data files

Implements DataWriter class for writing Iceberg data files as part of
issue #441 (task 2).

Implementation:
- Static factory method DataWriter::Make() for creating writer instances
- Support for Parquet and Avro file formats via WriterFactoryRegistry
- Complete DataFile metadata generation including partition info,
  column statistics, serialized bounds, and sort order ID
- Proper lifecycle management with Write/Close/Metadata methods
- Idempotent Close() - multiple calls succeed (no-op after first)
- PIMPL idiom for ABI stability
- Not thread-safe (documented)

Tests:
- 13 comprehensive unit tests including parameterized format tests
- Coverage: creation, write/close lifecycle, metadata generation,
  error handling, feature validation, and data size verification
- All tests passing (13/13)

Related to #441

Report URL: https://github.com/apache/iceberg-cpp/actions/runs/21771683504

With regards,
GitHub Actions via GitBox

Reply via email to