fix comments

apache · Feb 2, 2023 · 70f2503 · 70f2503
1 parent 9d6c4f2
commit 70f2503
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java b/nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java
@@ -103,7 +103,9 @@ public void initialize(String name, Map<String, String> options) {
       api = nessieClientBuilder.build(NessieApiV2.class);
     } else {
       throw new IllegalArgumentException(
-          "Unsupported client-api-version: " + apiVersion + ". Can only be v1 or v2");
+          String.format(
+              "Unsupported %s: " + apiVersion + ". Can only be v1 or v2",
+              NessieUtil.CLIENT_API_VERSION));
     }
 
     initialize(

diff --git a/nessie/src/test/java/org/apache/iceberg/nessie/TestNamespace.java b/nessie/src/test/java/org/apache/iceberg/nessie/TestNamespace.java
@@ -82,7 +82,7 @@ public void testListImplicitNamespaces() {
   public void testNamespacesWithV1Api() {
     createNamespacesAndTables();
 
-    testListTables();
+    verifyListTables();
 
     List<Namespace> namespaces;
     namespaces = catalog.listNamespaces(Namespace.of("foo"));
@@ -96,15 +96,15 @@ public void testNamespacesWithV1Api() {
   public void testNamespacesWithV2Api() {
     createNamespacesAndTables();
 
-    testListTables();
+    verifyListTables();
 
     List<Namespace> namespaces;
     namespaces = catalog.listNamespaces(Namespace.of("foo"));
     // NessieApiV2 will not list the implicit namespaces
     Assertions.assertThat(namespaces).isEqualTo(Collections.singletonList(Namespace.of("foo")));
   }
 
-  private void testListTables() {
+  private void verifyListTables() {
     List<TableIdentifier> tables = catalog.listTables(Namespace.of("foo"));
     Assertions.assertThat(tables).isNotNull().hasSize(2);
     tables = catalog.listTables(Namespace.of("foo", "bar"));