This is an automated email from the ASF dual-hosted git repository.
hellostephen 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 02200fce6d3 [test](mtmv) Fix regression test not stable and fail
(#52550)
02200fce6d3 is described below
commit 02200fce6d3d7f7f1fd925c62f62f30d3075b88c
Author: seawinde <[email protected]>
AuthorDate: Tue Jul 1 19:25:43 2025 +0800
[test](mtmv) Fix regression test not stable and fail (#52550)
### What problem does this PR solve?
grace_period.groovy(line 264) should not use sleep method, this would
calse unstable
part_partition_invalid(line 158) mv should not part in, not fail
---
.../nereids_rules_p0/mv/availability/grace_period.groovy | 6 ++++--
.../mv/external_table/part_partition_invalid.groovy | 10 ++++++----
2 files changed, 10 insertions(+), 6 deletions(-)
diff --git
a/regression-test/suites/nereids_rules_p0/mv/availability/grace_period.groovy
b/regression-test/suites/nereids_rules_p0/mv/availability/grace_period.groovy
index 1bc1de6bd9a..957ff1171c7 100644
---
a/regression-test/suites/nereids_rules_p0/mv/availability/grace_period.groovy
+++
b/regression-test/suites/nereids_rules_p0/mv/availability/grace_period.groovy
@@ -240,7 +240,7 @@ suite("grace_period") {
o_orderdate,
l_partkey,
l_suppkey""",
- 15,
+ 150000,
"l_shipdate")
sql """
@@ -308,7 +308,9 @@ suite("grace_period") {
l_suppkey;
""", mv_partition_allow_staleness_name)
sql "SET enable_materialized_view_rewrite=true"
- Thread.sleep(15000);
+
+ sql """ALTER MATERIALIZED VIEW ${mv_partition_allow_staleness_name}
set('grace_period'='0');"""
+
// after 10s when partition table, and query use the partition changed,
should fail
mv_not_part_in(
"""
diff --git
a/regression-test/suites/nereids_rules_p0/mv/external_table/part_partition_invalid.groovy
b/regression-test/suites/nereids_rules_p0/mv/external_table/part_partition_invalid.groovy
index 3c2ad76c81a..f0094c849b3 100644
---
a/regression-test/suites/nereids_rules_p0/mv/external_table/part_partition_invalid.groovy
+++
b/regression-test/suites/nereids_rules_p0/mv/external_table/part_partition_invalid.groovy
@@ -152,10 +152,11 @@ suite("part_partition_invalid",
"p0,external,external_docker") {
// refresh catalog cache
sql """ REFRESH CATALOG ${hive_catalog_name} PROPERTIES("invalid_cache" =
"true"); """
mv_rewrite_success(query_sql, mv_name)
+ // in 2.1 and 3.0 should not compensate union all, in 3.1 and master
should compensate union all
order_qt_after_modify_data_and_refresh_catalog """ ${query_sql}"""
- // query invalid partition data, should hit mv, because not check now.
- mv_rewrite_fail("""
+ // query invalid partition data, should not part in mv rewrite
+ mv_not_part_in("""
${query_sql} where o_orderdate = '2023-10-19';
""", mv_name)
order_qt_after_modify_and_refresh_catalog_19 """ ${query_sql} where
o_orderdate = '2023-10-19';"""
@@ -179,6 +180,7 @@ suite("part_partition_invalid",
"p0,external,external_docker") {
// refresh catalog cache
sql """ REFRESH CATALOG ${hive_catalog_name} PROPERTIES("invalid_cache" =
"true"); """
mv_rewrite_success(query_sql, mv_name)
+ // in 2.1 and 3.0 should not compensate union all, in 3.1 and master
should compensate union all
order_qt_after_add_data_with_refresh_catalog """ ${query_sql}"""
// query invalid partition data, should hit mv, because not check now.
@@ -188,8 +190,8 @@ suite("part_partition_invalid",
"p0,external,external_docker") {
order_qt_after_add_and_refresh_catalog_19 """ ${query_sql} where
o_orderdate = '2023-10-19';"""
- // query valid partition data, should hit mv
- mv_rewrite_fail("""
+ // query valid partition data, mv should not part in mv rewrite
+ mv_not_part_in("""
${query_sql} where o_orderdate = '2023-10-20';
""", mv_name)
order_qt_after_add_and_refresh_catalog_20 """ ${query_sql} where
o_orderdate = '2023-10-20';"""
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
