Bharath Vissapragada has submitted this change and it was merged.

Change subject: IMPALA-2093: Disallow NOT IN aggregate subqueries with a 
constant lhs expr
......................................................................


IMPALA-2093: Disallow NOT IN aggregate subqueries with a constant lhs expr

This commit fixes the issue where an incorrect rewrite is performed for the 
case of
uncorrelated aggregate subqueries participating in a NOT IN expr with a 
constant. With
this commit we detect cases like this and throw an AnalysisException.

Change-Id: Ia05ebc70ef310a0ec6f00ae6ae1c5de8fbd50240
Reviewed-on: http://gerrit.cloudera.org:8080/2007
Reviewed-by: Dimitris Tsirogiannis <[email protected]>
Tested-by: Internal Jenkins
(cherry picked from commit bf6fabd1029ffb3f087076a786ef9864509daef6)
Reviewed-on: http://gerrit.cloudera.org:8080/2158
Reviewed-by: Bharath Vissapragada <[email protected]>
Tested-by: Bharath Vissapragada <[email protected]>
---
M fe/src/main/java/com/cloudera/impala/analysis/StmtRewriter.java
M fe/src/test/java/com/cloudera/impala/analysis/AnalyzeSubqueriesTest.java
2 files changed, 17 insertions(+), 0 deletions(-)

Approvals:
  Bharath Vissapragada: Looks good to me, approved; Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/2158
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia05ebc70ef310a0ec6f00ae6ae1c5de8fbd50240
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.3.0_5.5.x
Gerrit-Owner: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Juan Yu <[email protected]>

Reply via email to