Clean up

Author: ajantha-bhat

core/src/main/java/org/apache/iceberg/SetPartitionStatistics.java

@@ -21,7 +21,6 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
-import org.apache.iceberg.relocated.com.google.common.base.Preconditions;
 import org.apache.iceberg.relocated.com.google.common.collect.Maps;
 import org.apache.iceberg.relocated.com.google.common.collect.Sets;
 
@@ -36,7 +35,10 @@ public SetPartitionStatistics(TableOperations ops) {
 
   @Override
   public UpdatePartitionStatistics setPartitionStatistics(PartitionStatisticsFile file) {
-    Preconditions.checkArgument(null != file, "partition statistics file must not be null");
+    if (file == null) {
+      return this;
+    }
+
     statsToSet.put(file.snapshotId(), file);
     return this;
   }

core/src/test/java/org/apache/iceberg/TestRowLineageMetadata.java

@@ -316,8 +316,9 @@ public void testEnableRowLineageViaPropertyAtTableCreation() {
             tableDir,
             "test",
             TEST_SCHEMA,
-            ImmutableMap.of(TableProperties.ROW_LINEAGE, "true"),
-            formatVersion);
+            PartitionSpec.unpartitioned(),
+            formatVersion,
+            ImmutableMap.of(TableProperties.ROW_LINEAGE, "true"));
     assertThat(table.ops().current().rowLineageEnabled()).isTrue();
   }
 

core/src/test/java/org/apache/iceberg/TestTables.java

@@ -52,26 +52,6 @@ public static TestTable create(
     return create(temp, name, schema, spec, SortOrder.unsorted(), formatVersion);
   }
 
-  public static TestTable create(
-      File temp, String name, Schema schema, Map<String, String> properties, int formatVersion) {
-    TestTableOperations ops = new TestTableOperations(name, temp);
-    if (ops.current() != null) {
-      throw new AlreadyExistsException("Table %s already exists at location: %s", name, temp);
-    }
-
-    ops.commit(
-        null,
-        newTableMetadata(
-            schema,
-            PartitionSpec.unpartitioned(),
-            SortOrder.unsorted(),
-            temp.toString(),
-            properties,
-            formatVersion));
-
-    return new TestTable(ops, name);
-  }
-
   public static TestTable create(
       File temp,
       String name,

data/src/main/java/org/apache/iceberg/data/PartitionStatsHandler.java

@@ -211,14 +211,12 @@ private static DataWriter<StructLike> dataWriter(
         return Parquet.writeData(outputFile)
             .schema(dataSchema)
             .createWriterFunc(InternalWriter::create)
-            .overwrite()
             .withSpec(PartitionSpec.unpartitioned())
             .build();
       case AVRO:
         return Avro.writeData(outputFile)
             .schema(dataSchema)
             .createWriterFunc(org.apache.iceberg.avro.InternalWriter::create)
-            .overwrite()
             .withSpec(PartitionSpec.unpartitioned())
             .build();
       case ORC: