Eduard Tudenhoefner
Eduard Tudenhoefner
This should be ready for review as Iceberg 1.5.0 has been officially released now: https://iceberg.apache.org/releases/#150-release
For parameterized testing we would need to have https://github.com/apache/iceberg/issues/9210 implemented first.
We should be mostly focusing on migrating Spark 3.5, which is already a big task
@chinmay-bhat I don't think this is complete yet. There are still plenty of places that use `org.junit.Test` and `@RunWith(Parameterized.class)` within the `iceberg-spark` module
@tomtongue yes that would be great if you could do that
@tomtongue It appears that there are still some references left to `org.junit.[Test|Assert|Assume]` in Spark 3.5 that we should fix as well.
We can also remove `SparkTestBase` and all of its subclasses, since those have all been migrated to JUnit5 now
Closing this as all of the Spark 3.5 tests have been migrated to JUnit5 + AssertJ. Thanks everyone for helping out and contributing
@elmehdibelgasmi would you be interested in opening up a PR and fixing this for S3FileIO?
Seems to be still an issue in the latest runs: https://github.com/apache/iceberg/actions/runs/8200693411/job/22427953001