Fix enum name in CountingAccessMetadata

The class name is "entity name", should generally be singular.
trinodb · May 5, 2023 · 0d409dd · 0d409dd
1 parent ee66294
commit 0d409dd
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 12 deletions.
diff --git a/core/trino-main/src/test/java/io/trino/metadata/CountingAccessMetadata.java b/core/trino-main/src/test/java/io/trino/metadata/CountingAccessMetadata.java
@@ -76,20 +76,20 @@
 public class CountingAccessMetadata
         implements Metadata
 {
-    public enum Methods
+    public enum Method
     {
         GET_TABLE_STATISTICS,
     }
 
     private final Metadata delegate;
-    private final ConcurrentHashMultiset<Methods> methodInvocations = ConcurrentHashMultiset.create();
+    private final ConcurrentHashMultiset<Method> methodInvocations = ConcurrentHashMultiset.create();
 
     public CountingAccessMetadata(Metadata delegate)
     {
         this.delegate = delegate;
     }
 
-    public Multiset<Methods> getMethodInvocations()
+    public Multiset<Method> getMethodInvocations()
     {
         return ImmutableMultiset.copyOf(methodInvocations);
     }
@@ -210,7 +210,7 @@ public TableMetadata getTableMetadata(Session session, TableHandle tableHandle)
     @Override
     public TableStatistics getTableStatistics(Session session, TableHandle tableHandle)
     {
-        methodInvocations.add(Methods.GET_TABLE_STATISTICS);
+        methodInvocations.add(Method.GET_TABLE_STATISTICS);
         return delegate.getTableStatistics(session, tableHandle);
     }
 

diff --git a/...ceberg/src/test/java/io/trino/plugin/iceberg/TestIcebergGetTableStatisticsOperations.java b/...ceberg/src/test/java/io/trino/plugin/iceberg/TestIcebergGetTableStatisticsOperations.java
@@ -116,8 +116,8 @@ public void testTwoWayJoin()
                 "FROM iceberg.tiny.orders o, iceberg.tiny.lineitem l " +
                 "WHERE o.orderkey = l.orderkey");
         assertThat(metadata.getMethodInvocations()).containsExactlyInAnyOrderElementsOf(
-                ImmutableMultiset.<CountingAccessMetadata.Methods>builder()
-                        .addCopies(CountingAccessMetadata.Methods.GET_TABLE_STATISTICS, 2)
+                ImmutableMultiset.<CountingAccessMetadata.Method>builder()
+                        .addCopies(CountingAccessMetadata.Method.GET_TABLE_STATISTICS, 2)
                         .build());
     }
 
@@ -128,8 +128,8 @@ public void testThreeWayJoin()
                 "FROM iceberg.tiny.customer c, iceberg.tiny.orders o, iceberg.tiny.lineitem l " +
                 "WHERE o.orderkey = l.orderkey AND c.custkey = o.custkey");
         assertThat(metadata.getMethodInvocations()).containsExactlyInAnyOrderElementsOf(
-                ImmutableMultiset.<CountingAccessMetadata.Methods>builder()
-                        .addCopies(CountingAccessMetadata.Methods.GET_TABLE_STATISTICS, 3)
+                ImmutableMultiset.<CountingAccessMetadata.Method>builder()
+                        .addCopies(CountingAccessMetadata.Method.GET_TABLE_STATISTICS, 3)
                         .build());
     }
 

diff --git a/testing/trino-tests/src/test/java/io/trino/tests/TestGetTableStatisticsOperations.java b/testing/trino-tests/src/test/java/io/trino/tests/TestGetTableStatisticsOperations.java
@@ -75,8 +75,8 @@ public void testTwoWayJoin()
                 "FROM tpch.tiny.orders o, tpch.tiny.lineitem l " +
                 "WHERE o.orderkey = l.orderkey");
         assertThat(metadata.getMethodInvocations()).containsExactlyInAnyOrderElementsOf(
-                ImmutableMultiset.<CountingAccessMetadata.Methods>builder()
-                        .addCopies(CountingAccessMetadata.Methods.GET_TABLE_STATISTICS, 2)
+                ImmutableMultiset.<CountingAccessMetadata.Method>builder()
+                        .addCopies(CountingAccessMetadata.Method.GET_TABLE_STATISTICS, 2)
                         .build());
     }
 
@@ -87,8 +87,8 @@ public void testThreeWayJoin()
                 "FROM tpch.tiny.customer c, tpch.tiny.orders o, tpch.tiny.lineitem l " +
                 "WHERE o.orderkey = l.orderkey AND c.custkey = o.custkey");
         assertThat(metadata.getMethodInvocations()).containsExactlyInAnyOrderElementsOf(
-                ImmutableMultiset.<CountingAccessMetadata.Methods>builder()
-                        .addCopies(CountingAccessMetadata.Methods.GET_TABLE_STATISTICS, 3)
+                ImmutableMultiset.<CountingAccessMetadata.Method>builder()
+                        .addCopies(CountingAccessMetadata.Method.GET_TABLE_STATISTICS, 3)
                         .build());
     }