rename to plural

apache · Dec 18, 2023 · e8335a0 · e8335a0
1 parent 577e242
commit e8335a0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/core/src/main/java/org/apache/iceberg/CatalogUtil.java b/core/src/main/java/org/apache/iceberg/CatalogUtil.java
@@ -120,7 +120,7 @@ public static void dropTableData(FileIO io, TableMetadata metadata) {
     deleteFiles(
         io,
         Iterables.transform(metadata.statisticsFiles(), StatisticsFile::path),
-        "statistic",
+        "statistics",
         true);
     deleteFile(io, metadata.metadataFileLocation(), "metadata");
   }

diff --git a/core/src/test/java/org/apache/iceberg/hadoop/TestCatalogUtilDropTable.java b/core/src/test/java/org/apache/iceberg/hadoop/TestCatalogUtilDropTable.java
@@ -67,10 +67,10 @@ public void dropTableDataDeletesExpectedFiles() throws IOException {
     Set<String> manifestLocations = manifestLocations(snapshotSet, table.io());
     Set<String> dataLocations = dataLocations(snapshotSet, table.io());
     Set<String> metadataLocations = metadataLocations(tableMetadata);
-    Set<String> statisticLocations = statisticLocations(tableMetadata);
+    Set<String> statsLocations = statsLocations(tableMetadata);
     Assertions.assertThat(manifestListLocations).as("should have 2 manifest lists").hasSize(2);
     Assertions.assertThat(metadataLocations).as("should have 4 metadata locations").hasSize(4);
-    Assertions.assertThat(statisticLocations).as("should have 1 statistic file").hasSize(1);
+    Assertions.assertThat(statsLocations).as("should have 1 stats file").hasSize(1);
 
     FileIO fileIO = Mockito.mock(FileIO.class);
     Mockito.when(fileIO.newInputFile(Mockito.anyString()))
@@ -90,7 +90,7 @@ public void dropTableDataDeletesExpectedFiles() throws IOException {
                     + manifestLocations.size()
                     + dataLocations.size()
                     + metadataLocations.size()
-                    + statisticLocations.size()))
+                    + statsLocations.size()))
         .deleteFile(argumentCaptor.capture());
 
     List<String> deletedPaths = argumentCaptor.getAllValues();
@@ -108,7 +108,7 @@ public void dropTableDataDeletesExpectedFiles() throws IOException {
         .containsAll(metadataLocations);
     Assertions.assertThat(deletedPaths)
         .as("should contain all created statistic")
-        .containsAll(statisticLocations);
+        .containsAll(statsLocations);
   }
 
   @Test
@@ -206,7 +206,7 @@ private Set<String> metadataLocations(TableMetadata tableMetadata) {
     return metadataLocations;
   }
 
-  private Set<String> statisticLocations(TableMetadata tableMetadata) {
+  private Set<String> statsLocations(TableMetadata tableMetadata) {
     return tableMetadata.statisticsFiles().stream()
         .map(StatisticsFile::path)
         .collect(Collectors.toSet());