diff --git a/core/src/test/java/org/apache/iceberg/MetadataTableScanTestBase.java b/core/src/test/java/org/apache/iceberg/MetadataTableScanTestBase.java index 7b06b6560958..ff9dfd1afc7f 100644 --- a/core/src/test/java/org/apache/iceberg/MetadataTableScanTestBase.java +++ b/core/src/test/java/org/apache/iceberg/MetadataTableScanTestBase.java @@ -21,8 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import java.io.IOException; -import java.util.Arrays; -import java.util.List; import java.util.Map; import java.util.Objects; import java.util.Set; @@ -38,11 +36,6 @@ @ExtendWith(ParameterizedTestExtension.class) public abstract class MetadataTableScanTestBase extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - protected Set scannedPaths(TableScan scan) { return StreamSupport.stream(scan.planFiles().spliterator(), false) .map(t -> t.file().location().toString()) diff --git a/core/src/test/java/org/apache/iceberg/TestBatchScans.java b/core/src/test/java/org/apache/iceberg/TestBatchScans.java index bc707e51af43..8c5ff22f40e7 100644 --- a/core/src/test/java/org/apache/iceberg/TestBatchScans.java +++ b/core/src/test/java/org/apache/iceberg/TestBatchScans.java @@ -21,7 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import java.io.IOException; -import java.util.Arrays; import java.util.List; import java.util.stream.Collectors; import org.apache.iceberg.io.CloseableIterable; @@ -32,11 +31,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestBatchScans extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @TestTemplate public void testDataTableScan() { table.newFastAppend().appendFile(FILE_A).appendFile(FILE_B).commit(); diff --git a/core/src/test/java/org/apache/iceberg/TestCreateTransaction.java b/core/src/test/java/org/apache/iceberg/TestCreateTransaction.java index 667c4b8cc133..79b99f1f61c8 100644 --- a/core/src/test/java/org/apache/iceberg/TestCreateTransaction.java +++ b/core/src/test/java/org/apache/iceberg/TestCreateTransaction.java @@ -24,8 +24,6 @@ import static org.assertj.core.api.Assertions.assertThatThrownBy; import java.io.IOException; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.exceptions.CommitFailedException; import org.apache.iceberg.relocated.com.google.common.collect.Lists; import org.apache.iceberg.relocated.com.google.common.collect.Sets; @@ -36,10 +34,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestCreateTransaction extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testCreateTransaction() throws IOException { diff --git a/core/src/test/java/org/apache/iceberg/TestEntriesMetadataTable.java b/core/src/test/java/org/apache/iceberg/TestEntriesMetadataTable.java index 51e5c8e6570e..25e138f96249 100644 --- a/core/src/test/java/org/apache/iceberg/TestEntriesMetadataTable.java +++ b/core/src/test/java/org/apache/iceberg/TestEntriesMetadataTable.java @@ -21,7 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assumptions.assumeThat; -import java.util.Arrays; import java.util.List; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableList; import org.apache.iceberg.relocated.com.google.common.collect.Iterables; @@ -32,11 +31,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestEntriesMetadataTable extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @TestTemplate public void testEntriesTable() { table.newAppend().appendFile(FILE_A).appendFile(FILE_B).commit(); diff --git a/core/src/test/java/org/apache/iceberg/TestFastAppend.java b/core/src/test/java/org/apache/iceberg/TestFastAppend.java index 6a5155b5d0f2..384e0edaaadd 100644 --- a/core/src/test/java/org/apache/iceberg/TestFastAppend.java +++ b/core/src/test/java/org/apache/iceberg/TestFastAppend.java @@ -23,7 +23,6 @@ import java.io.File; import java.io.IOException; -import java.util.Arrays; import java.util.List; import java.util.Objects; import java.util.Set; @@ -39,10 +38,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestFastAppend extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testAddManyFiles() { diff --git a/core/src/test/java/org/apache/iceberg/TestFindFiles.java b/core/src/test/java/org/apache/iceberg/TestFindFiles.java index 608358da3b49..883bf64781fc 100644 --- a/core/src/test/java/org/apache/iceberg/TestFindFiles.java +++ b/core/src/test/java/org/apache/iceberg/TestFindFiles.java @@ -21,7 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import java.util.Arrays; -import java.util.List; import java.util.Set; import org.apache.iceberg.expressions.Expressions; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableMap; @@ -34,10 +33,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestFindFiles extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testBasicBehavior() { diff --git a/core/src/test/java/org/apache/iceberg/TestIncrementalDataTableScan.java b/core/src/test/java/org/apache/iceberg/TestIncrementalDataTableScan.java index 359086cc52d9..f3a1410bf858 100644 --- a/core/src/test/java/org/apache/iceberg/TestIncrementalDataTableScan.java +++ b/core/src/test/java/org/apache/iceberg/TestIncrementalDataTableScan.java @@ -22,7 +22,6 @@ import static org.assertj.core.api.Assertions.assertThatThrownBy; import java.io.IOException; -import java.util.Arrays; import java.util.Collections; import java.util.List; import java.util.concurrent.Executors; @@ -41,10 +40,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestIncrementalDataTableScan extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @BeforeEach public void setupTableProperties() { diff --git a/core/src/test/java/org/apache/iceberg/TestLocationProvider.java b/core/src/test/java/org/apache/iceberg/TestLocationProvider.java index 7edba51c3d85..72967115ea24 100644 --- a/core/src/test/java/org/apache/iceberg/TestLocationProvider.java +++ b/core/src/test/java/org/apache/iceberg/TestLocationProvider.java @@ -21,7 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import java.util.Arrays; import java.util.List; import java.util.Map; import org.apache.iceberg.io.LocationProvider; @@ -32,10 +31,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestLocationProvider extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } // publicly visible for testing to be dynamically loaded public static class TwoArgDynamicallyLoadedLocationProvider implements LocationProvider { diff --git a/core/src/test/java/org/apache/iceberg/TestManifestCleanup.java b/core/src/test/java/org/apache/iceberg/TestManifestCleanup.java index 37ccbb8bb845..b177e13b36dd 100644 --- a/core/src/test/java/org/apache/iceberg/TestManifestCleanup.java +++ b/core/src/test/java/org/apache/iceberg/TestManifestCleanup.java @@ -20,18 +20,12 @@ import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.expressions.Expressions; import org.junit.jupiter.api.TestTemplate; import org.junit.jupiter.api.extension.ExtendWith; @ExtendWith(ParameterizedTestExtension.class) public class TestManifestCleanup extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testDelete() { diff --git a/core/src/test/java/org/apache/iceberg/TestManifestReaderStats.java b/core/src/test/java/org/apache/iceberg/TestManifestReaderStats.java index 9e1aa255d1a4..cd223212a4dd 100644 --- a/core/src/test/java/org/apache/iceberg/TestManifestReaderStats.java +++ b/core/src/test/java/org/apache/iceberg/TestManifestReaderStats.java @@ -23,8 +23,6 @@ import java.io.IOException; import java.nio.ByteBuffer; -import java.util.Arrays; -import java.util.List; import java.util.Map; import org.apache.iceberg.expressions.Expressions; import org.apache.iceberg.io.CloseableIterable; @@ -37,11 +35,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestManifestReaderStats extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - private static final Map VALUE_COUNT = ImmutableMap.of(3, 3L); private static final Map NULL_VALUE_COUNTS = ImmutableMap.of(3, 0L); private static final Map NAN_VALUE_COUNTS = ImmutableMap.of(3, 1L); diff --git a/core/src/test/java/org/apache/iceberg/TestManifestWriter.java b/core/src/test/java/org/apache/iceberg/TestManifestWriter.java index 7dd3ea2d1ce7..dea9f2d580d3 100644 --- a/core/src/test/java/org/apache/iceberg/TestManifestWriter.java +++ b/core/src/test/java/org/apache/iceberg/TestManifestWriter.java @@ -24,7 +24,6 @@ import java.io.File; import java.io.IOException; import java.io.UncheckedIOException; -import java.util.Arrays; import java.util.List; import java.util.UUID; import org.apache.iceberg.ManifestEntry.Status; @@ -36,11 +35,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestManifestWriter extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - private static final int FILE_SIZE_CHECK_ROWS_DIVISOR = 250; private static final long SMALL_FILE_SIZE = 10L; diff --git a/core/src/test/java/org/apache/iceberg/TestMicroBatchBuilder.java b/core/src/test/java/org/apache/iceberg/TestMicroBatchBuilder.java index d6faaba77709..a6f7ef02f7bb 100644 --- a/core/src/test/java/org/apache/iceberg/TestMicroBatchBuilder.java +++ b/core/src/test/java/org/apache/iceberg/TestMicroBatchBuilder.java @@ -20,7 +20,6 @@ import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; import java.util.Collections; import java.util.List; import org.apache.iceberg.MicroBatches.MicroBatch; @@ -32,10 +31,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestMicroBatchBuilder extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @BeforeEach public void setupTableProperties() { diff --git a/core/src/test/java/org/apache/iceberg/TestReplaceTransaction.java b/core/src/test/java/org/apache/iceberg/TestReplaceTransaction.java index c4d73b09db59..79196c0a7517 100644 --- a/core/src/test/java/org/apache/iceberg/TestReplaceTransaction.java +++ b/core/src/test/java/org/apache/iceberg/TestReplaceTransaction.java @@ -28,8 +28,6 @@ import java.io.File; import java.io.IOException; -import java.util.Arrays; -import java.util.List; import java.util.Map; import java.util.Set; import java.util.concurrent.atomic.AtomicInteger; @@ -47,10 +45,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestReplaceTransaction extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testReplaceTransactionWithCustomSortOrder() { diff --git a/core/src/test/java/org/apache/iceberg/TestRewriteManifests.java b/core/src/test/java/org/apache/iceberg/TestRewriteManifests.java index 532f51e3589d..0d5414eaf0d6 100644 --- a/core/src/test/java/org/apache/iceberg/TestRewriteManifests.java +++ b/core/src/test/java/org/apache/iceberg/TestRewriteManifests.java @@ -46,10 +46,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestRewriteManifests extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testRewriteManifestsAppendedDirectly() throws IOException { diff --git a/core/src/test/java/org/apache/iceberg/TestScanSummary.java b/core/src/test/java/org/apache/iceberg/TestScanSummary.java index 10c22b759988..0e23c8c57812 100644 --- a/core/src/test/java/org/apache/iceberg/TestScanSummary.java +++ b/core/src/test/java/org/apache/iceberg/TestScanSummary.java @@ -28,8 +28,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableList; import org.apache.iceberg.util.Pair; import org.junit.jupiter.api.TestTemplate; @@ -37,10 +35,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestScanSummary extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testSnapshotTimeRangeValidation() { diff --git a/core/src/test/java/org/apache/iceberg/TestSchemaAndMappingUpdate.java b/core/src/test/java/org/apache/iceberg/TestSchemaAndMappingUpdate.java index 111693792f78..dc6e9e1a7f85 100644 --- a/core/src/test/java/org/apache/iceberg/TestSchemaAndMappingUpdate.java +++ b/core/src/test/java/org/apache/iceberg/TestSchemaAndMappingUpdate.java @@ -22,8 +22,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import java.util.Arrays; -import java.util.List; import java.util.Objects; import java.util.Set; import org.apache.iceberg.exceptions.ValidationException; @@ -41,10 +39,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSchemaAndMappingUpdate extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testAddPrimitiveColumn() { diff --git a/core/src/test/java/org/apache/iceberg/TestSchemaID.java b/core/src/test/java/org/apache/iceberg/TestSchemaID.java index e3c58af1cbe3..4bbcadac2b17 100644 --- a/core/src/test/java/org/apache/iceberg/TestSchemaID.java +++ b/core/src/test/java/org/apache/iceberg/TestSchemaID.java @@ -23,7 +23,6 @@ import static org.assertj.core.api.Assertions.assertThat; import java.util.Arrays; -import java.util.List; import java.util.Map; import java.util.function.Function; import java.util.stream.Collectors; @@ -34,11 +33,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSchemaID extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @TestTemplate public void testNoChange() { int onlyId = table.schema().schemaId(); diff --git a/core/src/test/java/org/apache/iceberg/TestSetPartitionStatistics.java b/core/src/test/java/org/apache/iceberg/TestSetPartitionStatistics.java index 9504f78f5345..d43345ce3849 100644 --- a/core/src/test/java/org/apache/iceberg/TestSetPartitionStatistics.java +++ b/core/src/test/java/org/apache/iceberg/TestSetPartitionStatistics.java @@ -20,17 +20,11 @@ import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; -import java.util.List; import org.junit.jupiter.api.TestTemplate; import org.junit.jupiter.api.extension.ExtendWith; @ExtendWith(ParameterizedTestExtension.class) public class TestSetPartitionStatistics extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testEmptyUpdateStatistics() { diff --git a/core/src/test/java/org/apache/iceberg/TestSetStatistics.java b/core/src/test/java/org/apache/iceberg/TestSetStatistics.java index 4582e8f26c8a..e51614f45b2a 100644 --- a/core/src/test/java/org/apache/iceberg/TestSetStatistics.java +++ b/core/src/test/java/org/apache/iceberg/TestSetStatistics.java @@ -20,8 +20,6 @@ import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableList; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableMap; import org.junit.jupiter.api.TestTemplate; @@ -29,10 +27,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSetStatistics extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testEmptyUpdateStatistics() { diff --git a/core/src/test/java/org/apache/iceberg/TestSnapshot.java b/core/src/test/java/org/apache/iceberg/TestSnapshot.java index 59217e5b91c8..12bcd31a7d34 100644 --- a/core/src/test/java/org/apache/iceberg/TestSnapshot.java +++ b/core/src/test/java/org/apache/iceberg/TestSnapshot.java @@ -21,8 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assumptions.assumeThat; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.expressions.Expressions; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableSet; import org.apache.iceberg.relocated.com.google.common.collect.Iterables; @@ -31,10 +29,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSnapshot extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testAppendFilesFromTable() { diff --git a/core/src/test/java/org/apache/iceberg/TestSnapshotLoading.java b/core/src/test/java/org/apache/iceberg/TestSnapshotLoading.java index 96dde0f48eb7..72e7bc3b7114 100644 --- a/core/src/test/java/org/apache/iceberg/TestSnapshotLoading.java +++ b/core/src/test/java/org/apache/iceberg/TestSnapshotLoading.java @@ -24,7 +24,6 @@ import static org.mockito.Mockito.times; import static org.mockito.Mockito.verify; -import java.util.Arrays; import java.util.List; import java.util.function.Predicate; import java.util.stream.Collectors; @@ -41,11 +40,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSnapshotLoading extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - private Snapshot currentSnapshot; private List allSnapshots; private TableMetadata originalTableMetadata; diff --git a/core/src/test/java/org/apache/iceberg/TestSnapshotManager.java b/core/src/test/java/org/apache/iceberg/TestSnapshotManager.java index 61dd082701c2..bb7ccc0fd73a 100644 --- a/core/src/test/java/org/apache/iceberg/TestSnapshotManager.java +++ b/core/src/test/java/org/apache/iceberg/TestSnapshotManager.java @@ -21,8 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.exceptions.ValidationException; import org.apache.iceberg.relocated.com.google.common.collect.Iterables; import org.junit.jupiter.api.TestTemplate; @@ -49,11 +47,6 @@ public class TestSnapshotManager extends TestBase { .withRecordCount(1) .build(); - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @TestTemplate public void testCherryPickDynamicOverwrite() { table.newAppend().appendFile(FILE_A).commit(); diff --git a/core/src/test/java/org/apache/iceberg/TestSnapshotSelection.java b/core/src/test/java/org/apache/iceberg/TestSnapshotSelection.java index 6d339c0cbeaf..765b93574210 100644 --- a/core/src/test/java/org/apache/iceberg/TestSnapshotSelection.java +++ b/core/src/test/java/org/apache/iceberg/TestSnapshotSelection.java @@ -22,8 +22,6 @@ import java.nio.ByteBuffer; import java.nio.ByteOrder; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableMap; import org.apache.iceberg.relocated.com.google.common.collect.Iterables; import org.junit.jupiter.api.TestTemplate; @@ -31,10 +29,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSnapshotSelection extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testSnapshotSelectionById() { diff --git a/core/src/test/java/org/apache/iceberg/TestSnapshotSummary.java b/core/src/test/java/org/apache/iceberg/TestSnapshotSummary.java index c76ca45b534b..1eee2d293ec0 100644 --- a/core/src/test/java/org/apache/iceberg/TestSnapshotSummary.java +++ b/core/src/test/java/org/apache/iceberg/TestSnapshotSummary.java @@ -21,8 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assumptions.assumeThat; -import java.util.Arrays; -import java.util.List; import java.util.Map; import org.junit.jupiter.api.TestTemplate; import org.junit.jupiter.api.extension.ExtendWith; @@ -30,11 +28,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSnapshotSummary extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @TestTemplate public void testFileSizeSummary() { assertThat(listManifestFiles()).hasSize(0); diff --git a/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java b/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java index e802a0cf8246..82b3c4084da8 100644 --- a/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java +++ b/core/src/test/java/org/apache/iceberg/TestSplitPlanning.java @@ -24,7 +24,6 @@ import java.io.IOException; import java.nio.file.Path; -import java.util.Arrays; import java.util.List; import java.util.UUID; import java.util.stream.Collectors; @@ -54,11 +53,6 @@ public class TestSplitPlanning extends TestBase { private Table table = null; - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @Override @BeforeEach public void setupTable() throws IOException { diff --git a/core/src/test/java/org/apache/iceberg/TestTableMetadataSerialization.java b/core/src/test/java/org/apache/iceberg/TestTableMetadataSerialization.java index 94a3d35b35a6..eaa1c109f6f7 100644 --- a/core/src/test/java/org/apache/iceberg/TestTableMetadataSerialization.java +++ b/core/src/test/java/org/apache/iceberg/TestTableMetadataSerialization.java @@ -25,18 +25,12 @@ import java.io.ByteArrayOutputStream; import java.io.ObjectInputStream; import java.io.ObjectOutputStream; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.relocated.com.google.common.collect.Lists; import org.junit.jupiter.api.TestTemplate; import org.junit.jupiter.api.extension.ExtendWith; @ExtendWith(ParameterizedTestExtension.class) public class TestTableMetadataSerialization extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testSerialization() throws Exception { diff --git a/core/src/test/java/org/apache/iceberg/TestTableUpdatePartitionSpec.java b/core/src/test/java/org/apache/iceberg/TestTableUpdatePartitionSpec.java index f327ef752947..b1833684149d 100644 --- a/core/src/test/java/org/apache/iceberg/TestTableUpdatePartitionSpec.java +++ b/core/src/test/java/org/apache/iceberg/TestTableUpdatePartitionSpec.java @@ -23,8 +23,6 @@ import static org.apache.iceberg.expressions.Expressions.year; import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.transforms.Transforms; import org.apache.iceberg.types.Types; import org.junit.jupiter.api.BeforeEach; @@ -34,11 +32,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestTableUpdatePartitionSpec extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @BeforeEach public void verifyInitialSpec() { PartitionSpec initialSpec = PartitionSpec.builderFor(table.schema()).bucket("data", 16).build(); diff --git a/core/src/test/java/org/apache/iceberg/TestTimestampPartitions.java b/core/src/test/java/org/apache/iceberg/TestTimestampPartitions.java index 78842dbfe8bc..1d6b7dd3b751 100644 --- a/core/src/test/java/org/apache/iceberg/TestTimestampPartitions.java +++ b/core/src/test/java/org/apache/iceberg/TestTimestampPartitions.java @@ -23,18 +23,12 @@ import static org.assertj.core.api.Assertions.assertThat; import java.io.IOException; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.types.Types; import org.junit.jupiter.api.TestTemplate; import org.junit.jupiter.api.extension.ExtendWith; @ExtendWith(ParameterizedTestExtension.class) public class TestTimestampPartitions extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testPartitionAppend() throws IOException { diff --git a/core/src/test/java/org/apache/iceberg/TestTransaction.java b/core/src/test/java/org/apache/iceberg/TestTransaction.java index 363cbc52bcf2..6ea84e8fc2b2 100644 --- a/core/src/test/java/org/apache/iceberg/TestTransaction.java +++ b/core/src/test/java/org/apache/iceberg/TestTransaction.java @@ -24,7 +24,6 @@ import java.io.File; import java.io.IOException; -import java.util.Arrays; import java.util.List; import java.util.Set; import java.util.UUID; @@ -42,10 +41,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestTransaction extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testEmptyTransaction() { diff --git a/core/src/test/java/org/apache/iceberg/TestUpdatePartitionSpec.java b/core/src/test/java/org/apache/iceberg/TestUpdatePartitionSpec.java index 97f695315e16..9ee696b1e347 100644 --- a/core/src/test/java/org/apache/iceberg/TestUpdatePartitionSpec.java +++ b/core/src/test/java/org/apache/iceberg/TestUpdatePartitionSpec.java @@ -28,8 +28,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.transforms.Transforms; import org.apache.iceberg.types.Types; import org.junit.jupiter.api.TestTemplate; @@ -52,11 +50,6 @@ public class TestUpdatePartitionSpec extends TestBase { .bucket("id", 16, "shard") .build(); - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @TestTemplate public void testAddIdentityByName() { PartitionSpec updated = diff --git a/core/src/test/java/org/apache/iceberg/TestWapWorkflow.java b/core/src/test/java/org/apache/iceberg/TestWapWorkflow.java index b1e8e57850e8..02d42c81a196 100644 --- a/core/src/test/java/org/apache/iceberg/TestWapWorkflow.java +++ b/core/src/test/java/org/apache/iceberg/TestWapWorkflow.java @@ -21,8 +21,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.exceptions.CherrypickAncestorCommitException; import org.apache.iceberg.exceptions.DuplicateWAPCommitException; import org.apache.iceberg.exceptions.ValidationException; @@ -33,10 +31,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestWapWorkflow extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @BeforeEach public void setupTableProperties() { diff --git a/core/src/test/java/org/apache/iceberg/actions/TestSizeBasedRewriter.java b/core/src/test/java/org/apache/iceberg/actions/TestSizeBasedRewriter.java index 77d16d3bc821..74cd3ca4fe6e 100644 --- a/core/src/test/java/org/apache/iceberg/actions/TestSizeBasedRewriter.java +++ b/core/src/test/java/org/apache/iceberg/actions/TestSizeBasedRewriter.java @@ -20,7 +20,6 @@ import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; import java.util.List; import java.util.Map; import java.util.Set; @@ -28,7 +27,6 @@ import org.apache.iceberg.FileScanTask; import org.apache.iceberg.MockFileScanTask; import org.apache.iceberg.ParameterizedTestExtension; -import org.apache.iceberg.Parameters; import org.apache.iceberg.Table; import org.apache.iceberg.TestBase; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableList; @@ -39,11 +37,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestSizeBasedRewriter extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - @TestTemplate public void testSplitSizeLowerBound() { SizeBasedDataFileRewriterImpl rewriter = new SizeBasedDataFileRewriterImpl(table); diff --git a/core/src/test/java/org/apache/iceberg/io/TestOutputFileFactory.java b/core/src/test/java/org/apache/iceberg/io/TestOutputFileFactory.java index bb98925f504c..491e651c0099 100644 --- a/core/src/test/java/org/apache/iceberg/io/TestOutputFileFactory.java +++ b/core/src/test/java/org/apache/iceberg/io/TestOutputFileFactory.java @@ -20,11 +20,8 @@ import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.FileFormat; import org.apache.iceberg.ParameterizedTestExtension; -import org.apache.iceberg.Parameters; import org.apache.iceberg.PartitionKey; import org.apache.iceberg.PartitionSpec; import org.apache.iceberg.TestBase; @@ -38,11 +35,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestOutputFileFactory extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } - private static final int PARTITION_ID = 1; private static final int TASK_ID = 100; diff --git a/core/src/test/java/org/apache/iceberg/mapping/TestMappingUpdates.java b/core/src/test/java/org/apache/iceberg/mapping/TestMappingUpdates.java index 2161b40a60b5..d48ab7776896 100644 --- a/core/src/test/java/org/apache/iceberg/mapping/TestMappingUpdates.java +++ b/core/src/test/java/org/apache/iceberg/mapping/TestMappingUpdates.java @@ -21,10 +21,7 @@ import static org.apache.iceberg.types.Types.NestedField.required; import static org.assertj.core.api.Assertions.assertThat; -import java.util.Arrays; -import java.util.List; import org.apache.iceberg.ParameterizedTestExtension; -import org.apache.iceberg.Parameters; import org.apache.iceberg.TableProperties; import org.apache.iceberg.TestBase; import org.apache.iceberg.relocated.com.google.common.collect.ImmutableList; @@ -34,10 +31,6 @@ @ExtendWith(ParameterizedTestExtension.class) public class TestMappingUpdates extends TestBase { - @Parameters(name = "formatVersion = {0}") - protected static List parameters() { - return Arrays.asList(1, 2, 3); - } @TestTemplate public void testAddColumnMappingUpdate() {