This is an automated email from the ASF dual-hosted git repository.
morrysnow 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 9a30837d940 [enhance](mtmv)MTMV support hive table determine whether
the data is sync (#34845)
9a30837d940 is described below
commit 9a30837d940b6dfe8f13525e2086166e134d9c49
Author: zhangdong <[email protected]>
AuthorDate: Fri May 17 10:55:10 2024 +0800
[enhance](mtmv)MTMV support hive table determine whether the data is sync
(#34845)
Previously supported, this PR only turns on the switch
---
.../java/org/apache/doris/datasource/hive/HMSExternalTable.java | 2 +-
regression-test/suites/mtmv_p0/test_hive_mtmv.groovy | 8 --------
regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy | 8 ++++----
3 files changed, 5 insertions(+), 13 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 68cd9d374a5..d2eb0aa6e88 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
@@ -958,7 +958,7 @@ public class HMSExternalTable extends ExternalTable
implements MTMVRelatedTableI
@Override
public boolean needAutoRefresh() {
- return false;
+ return true;
}
@Override
diff --git a/regression-test/suites/mtmv_p0/test_hive_mtmv.groovy
b/regression-test/suites/mtmv_p0/test_hive_mtmv.groovy
index 8a91a45c5f8..872d7bf8ec0 100644
--- a/regression-test/suites/mtmv_p0/test_hive_mtmv.groovy
+++ b/regression-test/suites/mtmv_p0/test_hive_mtmv.groovy
@@ -61,14 +61,6 @@ suite("test_hive_mtmv",
"p0,external,hive,external_docker,external_docker_hive")
waitingMTMVTaskFinished(jobName)
order_qt_refresh_one_partition "SELECT * FROM ${mvName} order by
id"
- //refresh other partitions
- // current, for hive, auto refresh will not change data
- sql """
- REFRESH MATERIALIZED VIEW ${mvName} AUTO
- """
- waitingMTMVTaskFinished(jobName)
- order_qt_refresh_other_partition "SELECT * FROM ${mvName} order by
id"
-
//refresh complete
sql """
REFRESH MATERIALIZED VIEW ${mvName} complete
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 755c6c15b0d..ac19ecc0605 100644
--- a/regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy
+++ b/regression-test/suites/mtmv_p0/test_hive_refresh_mtmv.groovy
@@ -95,7 +95,7 @@ suite("test_hive_refresh_mtmv",
"p0,external,hive,external_docker,external_docke
assertTrue(showPartitionsResult.toString().contains("p_2020"))
sql """
- REFRESH MATERIALIZED VIEW ${mvName} complete
+ REFRESH MATERIALIZED VIEW ${mvName} auto
"""
def jobName = getJobName(dbName, mvName);
waitingMTMVTaskFinished(jobName)
@@ -108,7 +108,7 @@ suite("test_hive_refresh_mtmv",
"p0,external,hive,external_docker,external_docke
REFRESH catalog ${catalog_name}
"""
sql """
- REFRESH MATERIALIZED VIEW ${mvName} complete
+ REFRESH MATERIALIZED VIEW ${mvName} auto
"""
waitingMTMVTaskFinished(jobName)
order_qt_mtmv_2 "SELECT * FROM ${mvName} order by user_id"
@@ -124,7 +124,7 @@ suite("test_hive_refresh_mtmv",
"p0,external,hive,external_docker,external_docke
REFRESH catalog ${catalog_name}
"""
sql """
- REFRESH MATERIALIZED VIEW ${mvName} complete
+ REFRESH MATERIALIZED VIEW ${mvName} auto
"""
waitingMTMVTaskFinished(jobName)
showPartitionsResult = sql """show partitions from ${mvName}"""
@@ -143,7 +143,7 @@ suite("test_hive_refresh_mtmv",
"p0,external,hive,external_docker,external_docke
REFRESH catalog ${catalog_name}
"""
sql """
- REFRESH MATERIALIZED VIEW ${mvName} complete
+ REFRESH MATERIALIZED VIEW ${mvName} auto
"""
waitingMTMVTaskFinished(jobName)
showPartitionsResult = sql """show partitions from ${mvName}"""
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]