You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by yi...@apache.org on 2022/10/25 08:44:47 UTC
[doris] branch branch-1.1-lts updated: [fix](sort)order by constant expr bug (#13614)
This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-1.1-lts
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-1.1-lts by this push:
new 7e4b4dd79f [fix](sort)order by constant expr bug (#13614)
7e4b4dd79f is described below
commit 7e4b4dd79f1bb4deeda80c10152473606becf9e6
Author: starocean999 <40...@users.noreply.github.com>
AuthorDate: Tue Oct 25 16:44:42 2022 +0800
[fix](sort)order by constant expr bug (#13614)
---
.../src/main/java/org/apache/doris/analysis/SelectStmt.java | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/SelectStmt.java b/fe/fe-core/src/main/java/org/apache/doris/analysis/SelectStmt.java
index 4e98276954..2fe58db9ec 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/analysis/SelectStmt.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/SelectStmt.java
@@ -1409,7 +1409,15 @@ public class SelectStmt extends QueryStmt {
}
if (orderByElements != null) {
for (OrderByElement orderByElem : orderByElements) {
+ // we must make sure the expr is analyzed before rewrite
+ try {
+ orderByElem.getExpr().analyze(analyzer);
+ } catch (AnalysisException ex) {
+ //ignore any exception
+ }
orderByElem.setExpr(rewriter.rewrite(orderByElem.getExpr(), analyzer));
+ // after rewrite, need reset the analyze status for later re-analyze
+ orderByElem.getExpr().reset();
}
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org