This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 50815b1eb7c [opt](iceberg) no need to check the name format of 
iceberg's database (#32977)
50815b1eb7c is described below

commit 50815b1eb7c459f211f2396485f1825ee3753190
Author: Mingyu Chen <morning...@163.com>
AuthorDate: Fri Mar 29 00:25:49 2024 +0800

    [opt](iceberg) no need to check the name format of iceberg's database 
(#32977)
    
    No need to check the name format of iceberg's database.
    We should accept all databases.
---
 .../doris/datasource/iceberg/IcebergMetadataOps.java       | 14 +-------------
 1 file changed, 1 insertion(+), 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
index 767f4ce9101..0f43ef21c43 100644
--- 
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.analysis.DropTableStmt;
 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.ExternalCatalog;
 import org.apache.doris.datasource.ExternalDatabase;
@@ -72,16 +69,7 @@ public class IcebergMetadataOps implements 
ExternalMetadataOps {
 
     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());
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to