remove encode and then decode in rest util

apache · Nov 15, 2024 · 7bfa42d · 7bfa42d
1 parent e6a3e71
commit 7bfa42d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/polaris-service/src/main/java/org/apache/polaris/service/catalog/IcebergCatalogAdapter.java b/polaris-service/src/main/java/org/apache/polaris/service/catalog/IcebergCatalogAdapter.java
@@ -180,7 +180,7 @@ public Response loadNamespaceMetadata(
   }
 
   private static Namespace decodeNamespace(String namespace) {
-    return RESTUtil.decodeNamespace(URLEncoder.encode(namespace, Charset.defaultCharset()));
+    return RESTUtil.decodeNamespace(namespace);
   }
 
   @Override

diff --git a/...s-service/src/test/java/org/apache/polaris/service/PolarisApplicationIntegrationTest.java b/...s-service/src/test/java/org/apache/polaris/service/PolarisApplicationIntegrationTest.java
@@ -341,7 +341,7 @@ public void testIcebergListNamespacesNestedNotFound() throws IOException {
                   sessionCatalog.listNamespaces(
                       sessionContext, Namespace.of("top_level", "whoops")))
           .isInstanceOf(NoSuchNamespaceException.class)
-          .hasMessage("Namespace does not exist: top_level%1Fwhoops");
+          .hasMessage("Namespace does not exist: top_level.whoops");
     }
   }