The GitHub Actions job "Flink CI" on iceberg.git/feature/rewrite-data-files-branch-support has succeeded. Run started by GitHub user 97harsh (triggered by 97harsh).
Head commit for run: 975e6e797feeadca565750e68b74a4a69b27acc9 / Harsh Sharma <[email protected]> Fix constructor chain in RewriteDataFilesCommitManager to prevent NPE The previous implementation incorrectly passed null as a 4th parameter in the 2-arg and 3-arg constructors, which caused them to call the wrong constructor overload. This resulted in snapshotProperties being null, leading to a NullPointerException when commitFileGroups() tried to iterate over properties with forEach(). The issue broke Flink maintenance API tests (TestRewriteDataFiles and TestFlinkTableSinkCompaction) because the Flink DataFileRewriteCommitter uses the 2-arg constructor. Files were not being rewritten as expected. Changes: - Line 51: Remove null parameter to call 3-arg constructor - Line 56: Remove null parameter to call 4-arg constructor with Map This ensures the constructor chain properly passes through existing constructors without introducing null values, and branch parameter is correctly passed only through the appropriate constructors. Report URL: https://github.com/apache/iceberg/actions/runs/20715566053 With regards, GitHub Actions via GitBox
