address feedback

Author: gh-yzou

plugins/spark/v3.5/spark/src/main/java/org/apache/polaris/spark/PolarisRESTCatalog.java

@@ -152,9 +152,10 @@ public void close() throws IOException {
 
   @Override
   public List<TableIdentifier> listGenericTables(Namespace ns) {
-    if (!endpoints.contains(PolarisEndpoints.V1_LIST_GENERIC_TABLES)) {
-      return ImmutableList.of();
-    }
+    Endpoint.check(endpoints, PolarisEndpoints.V1_LIST_GENERIC_TABLES);
+    // if (!endpoints.contains(PolarisEndpoints.V1_LIST_GENERIC_TABLES)) {
+    //  return ImmutableList.of();
+    // }
 
     Map<String, String> queryParams = Maps.newHashMap();
     ImmutableList.Builder<TableIdentifier> tables = ImmutableList.builder();

plugins/spark/v3.5/spark/src/main/java/org/apache/polaris/spark/PolarisSparkCatalog.java

@@ -114,8 +114,12 @@ public void renameTable(Identifier from, Identifier to)
 
   @Override
   public Identifier[] listTables(String[] namespace) {
-    return this.polarisCatalog.listGenericTables(Namespace.of(namespace)).stream()
-        .map(ident -> Identifier.of(ident.namespace().levels(), ident.name()))
-        .toArray(Identifier[]::new);
+    try {
+      return this.polarisCatalog.listGenericTables(Namespace.of(namespace)).stream()
+          .map(ident -> Identifier.of(ident.namespace().levels(), ident.name()))
+          .toArray(Identifier[]::new);
+    } catch (UnsupportedOperationException ex) {
+      return new Identifier[0];
+    }
   }
 }