The GitHub Actions job "Flink CI" on iceberg.git/DeprecateSnapshotIOMethods has 
failed.
Run started by GitHub user RussellSpitzer (triggered by RussellSpitzer).

Head commit for run:
86d4f2e14bb9d5cf393f98304d32f2c84fee2d86 / Russell Spitzer 
<[email protected]>
API, CORE, Flink, Spark: Deprecate Snapshot Change Methods

We currently offer several methods for getting files changed in
a snapshot but they rely on the assumption that you can read the
partition_spec from the manifest metadata. In advance of the
move to Parquet Manifest, we'll be no longer able to rely on
this part of the manifest read code.

In this PR we deprecate those existing methods and create a
new utility class which can do the same thing as the old
Snapshot methods. The new utility class does not assume
that the manifest read code can actually read the partition_spec
info and instead takes it as an arguement.

In production code there are only a small number of actual
uses

1. CherryPickOperation
2. MicroBatches

Within our other modules we also had a few usages

Flink
1. TableChange

Spark
2. MicroBatchStream

Unfortuantely there are also a huge number of test usages of these
methods, the majority of this commit is cleaning those up.

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

With regards,
GitHub Actions via GitBox

Reply via email to