Thanks for your contribution! On Fri, Oct 21, 2016 at 12:45 AM, Internal Jenkins (Code Review) < [email protected]> wrote:
> Internal Jenkins has submitted this change and it was merged. > > Change subject: IMPALA-4325: StmtRewrite lost parentheses of > CompoundPredicate > ...................................................................... > > > IMPALA-4325: StmtRewrite lost parentheses of CompoundPredicate > > StmtRewrite lost parentheses of CompoundPredicate in > pushNegationToOperands() > and leads to incorrect toSql() result. Even though this issue would not > leads > to incorrect result of query, it makes user confuse of the logical operator > precedence of predicates shown in EXPLAIN statement. > > Change-Id: I79bfc67605206e0e026293bf7032a88227a95623 > Reviewed-on: http://gerrit.cloudera.org:8080/4753 > Reviewed-by: Alex Behm <[email protected]> > Tested-by: Internal Jenkins > --- > M fe/src/main/java/org/apache/impala/analysis/Expr.java > M testdata/workloads/functional-planner/queries/PlannerTest/ > subquery-rewrite.test > 2 files changed, 28 insertions(+), 1 deletion(-) > > Approvals: > Internal Jenkins: Verified > Alex Behm: Looks good to me, approved > > > > -- > To view, visit http://gerrit.cloudera.org:8080/4753 > To unsubscribe, visit http://gerrit.cloudera.org:8080/settings > > Gerrit-MessageType: merged > Gerrit-Change-Id: I79bfc67605206e0e026293bf7032a88227a95623 > Gerrit-PatchSet: 6 > Gerrit-Project: Impala-ASF > Gerrit-Branch: master > Gerrit-Owner: Yuanhao Luo <[email protected]> > Gerrit-Reviewer: Alex Behm <[email protected]> > Gerrit-Reviewer: Internal Jenkins > Gerrit-Reviewer: Yuanhao Luo <[email protected]> > > -- > You received this message because you are subscribed to the Google Groups > "impala-cr" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > For more options, visit https://groups.google.com/a/cloudera.com/d/optout. >
