The GitHub Actions job "Java CI" on iceberg.git/rewrite_data_files2 has 
succeeded.
Run started by GitHub user shangxinli (triggered by shangxinli).

Head commit for run:
5339589134f48bdc9980609b4d966468083773f1 / Xinli Shang <[email protected]>
Address review feedback

- Add static constants for row lineage column types and descriptors
- Extract writer() helper method to reduce code duplication
- Rename methods for clarity:
  - mergeFilesWithSchema -> binaryMerge
  - mergeFilesWithRowLineageAndSchema -> generateRowLineageAndMerge
  - validateRowLineageColumnsHaveNoNulls -> allRowLineageColumnsNonNull
- Use Statistics builder instead of manual LongStatistics construction
- Add try-with-resources for ValuesWriter
- Fix encoding parameters: BIT_PACKED -> RLE for definition/repetition levels
- Change dataFile.path().toString() to dataFile.location()
- Use Java 9 try-with-resources syntax for readSchema()
- Remove redundant comments and simplify JavaDoc
- Remove InternalFileEncryptor import (no longer needed)
- Add BytesUtils import for Statistics builder

Minor formatting update

Fix spotless formatting

Report URL: https://github.com/apache/iceberg/actions/runs/20400052101

With regards,
GitHub Actions via GitBox

Reply via email to