Hello Quanlong Huang, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/19049

to look at the new patch set (#3).

Change subject: IMPALA-11612: fix ORDER BY expression rewrite bug
......................................................................

IMPALA-11612: fix ORDER BY expression rewrite bug

When the query SQL is overwritten, both the selectList and groupByList
fields are preprocessed except for the orderByList fields, the result
of rewoten orderByList is not same as selectList and groupByList.
It occur exception in method SelectStmt.verifyAggregation.
So we should add preprocessing before the orderByList be rewrote.

Change-Id: I438c61c647e3f99665c94ea15e3a9d98b9ff9912
---
M fe/src/main/java/org/apache/impala/analysis/SelectStmt.java
M fe/src/test/java/org/apache/impala/analysis/ExprRewriterTest.java
2 files changed, 20 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/49/19049/3
--
To view, visit http://gerrit.cloudera.org:8080/19049
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I438c61c647e3f99665c94ea15e3a9d98b9ff9912
Gerrit-Change-Number: 19049
Gerrit-PatchSet: 3
Gerrit-Owner: Anonymous Coward <jhkc...@qq.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>

Reply via email to