Hello Aman Sinha, Kurt Deschler, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17200
to look at the new patch set (#4).
Change subject: IMPALA-10593: Conditionally skip runtime filter for outer joins
......................................................................
IMPALA-10593: Conditionally skip runtime filter for outer joins
Currently there is code that asserts that an Expr is not constant after
substituting SlotRefs with constant nulls.
For External FE, this restriction to be weakened. In a case where
an Expr is checked and the Expr is not constant even after substituting
nulls, the result will be to not generate a runtime filter for that Expr.
Testing:
Manually tested with this query in the External FE:
select id, int_col, year, month from alltypessmall s
where s.int_col = (select count(*) from alltypestiny t where s.id = t.id)
order by id
Change-Id: I46462e2030731d97c4c88e364148c0093c025ab3
---
M fe/src/main/java/org/apache/impala/analysis/Analyzer.java
M fe/src/main/java/org/apache/impala/planner/RuntimeFilterGenerator.java
2 files changed, 8 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/00/17200/4
--
To view, visit http://gerrit.cloudera.org:8080/17200
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I46462e2030731d97c4c88e364148c0093c025ab3
Gerrit-Change-Number: 17200
Gerrit-PatchSet: 4
Gerrit-Owner: Steve Carlin <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>
