This is an automated email from the ASF dual-hosted git repository.
starocean999 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 2ec33950879 [fix](planner)the data type should be the same between
input slot and sort slot (#27137)
2ec33950879 is described below
commit 2ec33950879128ef6a5aeca2fa084e4346fc05b9
Author: starocean999 <[email protected]>
AuthorDate: Thu Nov 23 18:40:02 2023 +0800
[fix](planner)the data type should be the same between input slot and sort
slot (#27137)
---
.../java/org/apache/doris/planner/AnalyticPlanner.java | 5 +++--
.../test_inlineview_with_window_function.out | 6 ++++++
.../test_inlineview_with_window_function.groovy | 16 ++++++++++++++++
3 files changed, 25 insertions(+), 2 deletions(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/planner/AnalyticPlanner.java
b/fe/fe-core/src/main/java/org/apache/doris/planner/AnalyticPlanner.java
index 433fe76fd9e..7687b5812a4 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/planner/AnalyticPlanner.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/planner/AnalyticPlanner.java
@@ -313,9 +313,10 @@ public class AnalyticPlanner {
SlotDescriptor sortSlotDesc =
analyzer.getDescTbl().addSlotDescriptor(sortTupleDesc);
if (inputSlotDesc.getColumn() != null) {
sortSlotDesc.setColumn(inputSlotDesc.getColumn());
- } else {
- sortSlotDesc.setType(inputSlotDesc.getType());
}
+ // always set type as inputSlotDesc's type
+ sortSlotDesc.setType(inputSlotDesc.getType());
+
// all output slots need to be materialized
sortSlotDesc.setIsMaterialized(true);
sortSlotDesc.setIsNullable(inputSlotDesc.getIsNullable());
diff --git
a/regression-test/data/correctness_p0/test_inlineview_with_window_function.out
b/regression-test/data/correctness_p0/test_inlineview_with_window_function.out
index 281bf5dd1b2..778d5d8c716 100644
---
a/regression-test/data/correctness_p0/test_inlineview_with_window_function.out
+++
b/regression-test/data/correctness_p0/test_inlineview_with_window_function.out
@@ -2,3 +2,9 @@
-- !order --
2023-06-10 cib2205045_1_1s 0.0000 168939.0 0.0000 0.0 0.0000
0.0 0.0000 day
+-- !order2 --
+1 1 1
+2 2 2
+3 3 3
+4 4 4
+
diff --git
a/regression-test/suites/correctness_p0/test_inlineview_with_window_function.groovy
b/regression-test/suites/correctness_p0/test_inlineview_with_window_function.groovy
index cdeb65e9ec5..1d3c7fdf58f 100644
---
a/regression-test/suites/correctness_p0/test_inlineview_with_window_function.groovy
+++
b/regression-test/suites/correctness_p0/test_inlineview_with_window_function.groovy
@@ -151,4 +151,20 @@ suite("test_inlineview_with_window_function") {
group by ordernum
)tmp1
on tmp.ordernum=tmp1.ordernum;"""
+
+ sql """set enable_nereids_planner=false;"""
+ qt_order2 """
+ SELECT
+ row_number() over(partition by add_date order by pc_num desc)
+ ,row_number() over(partition by add_date order by vc_num desc)
+ ,row_number() over(partition by add_date order by vt_num desc)
+ FROM (
+ SELECT
+ cast(dnt as datev2) add_date
+ ,row_number() over(order by dnt) pc_num
+ ,row_number() over(order by dnt) vc_num
+ ,row_number() over(order by dnt) vt_num
+ FROM test_table_aaa
+ ) t;
+ """
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
