Michael Smith has uploaded this change for review. ( http://gerrit.cloudera.org:8080/21729
Change subject: IMPALA-13185: Include runtime filter source in key ...................................................................... IMPALA-13185: Include runtime filter source in key Incorporates the build-side PlanNode of a runtime filter in the tuple cache key to avoid re-using intermediate results that were generated using a runtime filter on the same target but different selection criteria (build-side conjuncts). We currently don't support caching ExchangeNode, but a common scenario is a runtime filter produced by a HashJoin, with an Exchange on the build side. Looks through the first ExchangeNode when considering the cache key and eligibility for the build side source for a runtime filter. Testing shows all tests now passing for test_tuple_cache_tpc_queries except those that hit "TupleCacheNode does not enforce limits itself and cannot have a limit set." Change-Id: I0077964be5acdb588d76251a6a39e57a0f42bb5a --- M fe/src/main/java/org/apache/impala/planner/JoinNode.java M fe/src/main/java/org/apache/impala/planner/PlanNode.java M fe/src/main/java/org/apache/impala/planner/RuntimeFilterGenerator.java M fe/src/main/java/org/apache/impala/planner/TupleCacheInfo.java 4 files changed, 67 insertions(+), 26 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/29/21729/1 -- To view, visit http://gerrit.cloudera.org:8080/21729 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I0077964be5acdb588d76251a6a39e57a0f42bb5a Gerrit-Change-Number: 21729 Gerrit-PatchSet: 1 Gerrit-Owner: Michael Smith <[email protected]>
