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 94d98c4c525 [fix](test) use different catalog name for 
test_hive_parquet_skip_page (#46315)
94d98c4c525 is described below

commit 94d98c4c525071a79d0c7586afc9bd90b4e06d1e
Author: Mingyu Chen (Rayner) <morning...@163.com>
AuthorDate: Fri Jan 3 10:34:45 2025 +0800

    [fix](test) use different catalog name for test_hive_parquet_skip_page 
(#46315)
    
    ### What problem does this PR solve?
    
    Otherwise it may cause other test case fail because it will drop that
    catalog.
---
 .../src/main/java/org/apache/doris/nereids/util/RelationUtil.java      | 3 ++-
 .../suites/external_table_p0/hive/test_hive_parquet_skip_page.groovy   | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/util/RelationUtil.java 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/util/RelationUtil.java
index bba252c3b04..46c594d4994 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/util/RelationUtil.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/util/RelationUtil.java
@@ -60,7 +60,8 @@ public class RelationUtil {
                 String tableName = nameParts.get(0);
                 CatalogIf catalogIf = context.getCurrentCatalog();
                 if (catalogIf == null) {
-                    throw new IllegalStateException("Current catalog is not 
set.");
+                    throw new IllegalStateException(
+                            "Current catalog is not set. default catalog is: " 
+ context.getDefaultCatalog());
                 }
                 String catalogName = catalogIf.getName();
                 String dbName = context.getDatabase();
diff --git 
a/regression-test/suites/external_table_p0/hive/test_hive_parquet_skip_page.groovy
 
b/regression-test/suites/external_table_p0/hive/test_hive_parquet_skip_page.groovy
index 67e594b21f1..ebdbedf139d 100644
--- 
a/regression-test/suites/external_table_p0/hive/test_hive_parquet_skip_page.groovy
+++ 
b/regression-test/suites/external_table_p0/hive/test_hive_parquet_skip_page.groovy
@@ -97,7 +97,7 @@ suite("test_hive_parquet_skip_page", 
"p0,external,hive,external_docker,external_
     for (String hivePrefix : ["hive2", "hive3"]) {
         try {
             String hms_port = context.config.otherConfigs.get(hivePrefix + 
"HmsPort")
-            String catalog_name = "${hivePrefix}_test_parquet"
+            String catalog_name = "${hivePrefix}_test_parquet_skip_page"
             String externalEnvIp = 
context.config.otherConfigs.get("externalEnvIp")
 
             sql """drop catalog if exists ${catalog_name}"""


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

Reply via email to