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

dataroaring pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new e71c5c0bb8f branch-3.0:[opt](mtmv) metadata cache of external table no 
longer be refreshed b… (#48767)
e71c5c0bb8f is described below

commit e71c5c0bb8f51b4958b4bc8e651efe5f8a3c1a41
Author: zhangdong <zhangd...@selectdb.com>
AuthorDate: Mon Mar 10 17:34:10 2025 +0800

    branch-3.0:[opt](mtmv) metadata cache of external table no longer be 
refreshed b… (#48767)
    
    …efore run async mv task (#48172)
    
    pick: https://github.com/apache/doris/pull/48172
---
 .../org/apache/doris/datasource/hive/HMSExternalTable.java   |  2 --
 regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy | 12 ++++++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HMSExternalTable.java
 
b/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HMSExternalTable.java
index b554f508103..cb999d79823 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HMSExternalTable.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/datasource/hive/HMSExternalTable.java
@@ -1024,8 +1024,6 @@ public class HMSExternalTable extends ExternalTable 
implements MTMVRelatedTableI
 
     @Override
     public void beforeMTMVRefresh(MTMV mtmv) throws DdlException {
-        Env.getCurrentEnv().getRefreshManager()
-                .refreshTable(getCatalog().getName(), getDbName(), getName(), 
true);
     }
 
     public HoodieTableMetaClient getHudiClient() {
diff --git a/regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy 
b/regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy
index 75ec61a518b..f2cfae8c3e9 100644
--- a/regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy
+++ b/regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy
@@ -104,6 +104,9 @@ suite("test_hive_refresh_mtmv", 
"p0,external,hive,external_docker,external_docke
         // hive data change
         logger.info("hive sql: " + insert_str)
         hive_docker """ ${insert_str} """
+        sql """
+                REFRESH catalog ${catalog_name}
+            """
         sql """
             REFRESH MATERIALIZED VIEW ${mvName} auto
         """
@@ -117,6 +120,9 @@ suite("test_hive_refresh_mtmv", 
"p0,external,hive,external_docker,external_docke
                                     """
         logger.info("hive sql: " + add_partition2021_str)
         hive_docker """ ${add_partition2021_str} """
+        sql """
+                REFRESH catalog ${catalog_name}
+            """
         sql """
             REFRESH MATERIALIZED VIEW ${mvName} auto
         """
@@ -133,6 +139,9 @@ suite("test_hive_refresh_mtmv", 
"p0,external,hive,external_docker,external_docke
                                         """
         logger.info("hive sql: " + drop_partition2021_str)
         hive_docker """ ${drop_partition2021_str} """
+        sql """
+                REFRESH catalog ${catalog_name}
+            """
             sql """
                 REFRESH MATERIALIZED VIEW ${mvName} auto
             """
@@ -148,6 +157,9 @@ suite("test_hive_refresh_mtmv", 
"p0,external,hive,external_docker,external_docke
                                     """
         logger.info("hive sql: " + rename_column_str)
         hive_docker """ ${rename_column_str} """
+        sql """
+                REFRESH catalog ${catalog_name}
+            """
             sql """
                 REFRESH MATERIALIZED VIEW ${mvName} complete
             """


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

Reply via email to