impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dimitris Tsirogiannis (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5856: Fix outer join predicate assignment.
Date Wed, 13 Sep 2017 05:01:49 GMT
Dimitris Tsirogiannis has posted comments on this change.

Change subject: IMPALA-5856: Fix outer join predicate assignment.

Patch Set 1:

File fe/src/main/java/org/apache/impala/analysis/

PS1, Line 1381: // Right-most full-outer join table ref that is equal to or to the left of
              :     // table ref materializing 'tids'.
Trying to visualize this hurts my brain :) Not sure if it helps to understand the logic in
this function. The comment in L1385 should suffice.

PS1, Line 1385: this
What is "this" refer to? After reading this more carefully, I think I understand what it means.
I think it is better to move the comment below L1388.

PS1, Line 1398: globalState_.ojClauseByConjunct.get(e.getId());
File testdata/workloads/functional-planner/queries/PlannerTest/outer-joins.test:

PS1, Line 1020: from the On-clause of a left outer join
Out of curiosity, did you check if we have the same issue if there is a left outer join followed
by a full outer join? If we don't have a test like that, maybe add it for completion.

Line 1028:      and t1.bigint_col > 10 and t2.bigint_col > 30
To make it more interesting, add a where clause with one predicate that references t1 and
t2 and one that references t3.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I93db34d988cb66e00aa05d7dc161e0ca47042acb
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Alex Behm <>
Gerrit-Reviewer: Dimitris Tsirogiannis <>
Gerrit-HasComments: Yes

View raw message