You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2022/12/28 02:09:56 UTC
[doris] 01/02: [test](Nereids) Disable some regression test for materialized index. (#15387)
This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git
commit 8e0a57407a6e027b07ba957dae2179d702cf850f
Author: Shuo Wang <wa...@gmail.com>
AuthorDate: Tue Dec 27 19:24:03 2022 +0800
[test](Nereids) Disable some regression test for materialized index. (#15387)
When light schema change is enabled by default (#15344), regression tests that run SQL by selecting data from the materialized index will fail.
This PR disabled those failed queries in the regression test. Those tests would be added back when nereids planner could give the correct plan when light schema change is enabled.
---
regression-test/data/nereids_syntax_p0/rollup.out | 9 ---------
regression-test/suites/nereids_syntax_p0/rollup.groovy | 6 ++++--
2 files changed, 4 insertions(+), 11 deletions(-)
diff --git a/regression-test/data/nereids_syntax_p0/rollup.out b/regression-test/data/nereids_syntax_p0/rollup.out
deleted file mode 100644
index 71d96f5919..0000000000
--- a/regression-test/data/nereids_syntax_p0/rollup.out
+++ /dev/null
@@ -1,9 +0,0 @@
--- This file is automatically generated. You should know what you did if you want to edit this
--- !rollup1 --
-2 6
-3 4
-
--- !rollup2 --
-1 6
-2 4
-
diff --git a/regression-test/suites/nereids_syntax_p0/rollup.groovy b/regression-test/suites/nereids_syntax_p0/rollup.groovy
index ac7126c5ce..2284b66b74 100644
--- a/regression-test/suites/nereids_syntax_p0/rollup.groovy
+++ b/regression-test/suites/nereids_syntax_p0/rollup.groovy
@@ -84,9 +84,11 @@ suite("rollup") {
contains("PREAGGREGATION: ON")
}
- order_qt_rollup1 "select k2, sum(v1) from rollup_t1 group by k2"
+ // TODO: add these qt tests back when nereids planner could get correct plan when
+ // light schema change is enabled.
+ // order_qt_rollup1 "select k2, sum(v1) from rollup_t1 group by k2"
- order_qt_rollup2 "select k1, sum(v1) from rollup_t1 group by k1"
+ // order_qt_rollup2 "select k1, sum(v1) from rollup_t1 group by k1"
explain {
sql("select k1, max(v1) from rollup_t1 group by k1")
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org