[opt](iceberg) no need to check the name format of iceberg's database (…

…apache#32977) No need to check the name format of iceberg's database. We should accept all databases.
morningman · Apr 7, 2024 · 8bb2ef1 · 8bb2ef1
1 parent e9b67bc
commit 8bb2ef1
Showing 1 changed file with 1 addition and 13 deletions.
diff --git a/fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/IcebergMetadataOps.java b/fe/fe-core/src/main/java/org/apache/doris/datasource/iceberg/IcebergMetadataOps.java
@@ -24,11 +24,8 @@
 import org.apache.doris.catalog.Column;
 import org.apache.doris.catalog.StructField;
 import org.apache.doris.catalog.StructType;
-import org.apache.doris.common.AnalysisException;
 import org.apache.doris.common.DdlException;
-import org.apache.doris.common.FeNameFormat;
 import org.apache.doris.common.UserException;
-import org.apache.doris.common.util.Util;
 import org.apache.doris.datasource.DorisTypeVisitor;
 import org.apache.doris.datasource.ExternalDatabase;
 import org.apache.doris.datasource.operations.ExternalMetadataOps;
@@ -71,16 +68,7 @@ public boolean tableExist(String dbName, String tblName) {
 
     public List<String> listDatabaseNames() {
         return nsCatalog.listNamespaces().stream()
-                .map(e -> {
-                    String dbName = e.toString();
-                    try {
-                        FeNameFormat.checkDbName(dbName);
-                    } catch (AnalysisException ex) {
-                        Util.logAndThrowRuntimeException(LOG,
-                                String.format("Not a supported namespace name format: %s", dbName), ex);
-                    }
-                    return dbName;
-                })
+                .map(e -> e.toString())
                 .collect(Collectors.toList());
     }