Hello Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21857
to look at the new patch set (#2).
Change subject: IMPALA-13181: Disable tuple caching for locations with limits
......................................................................
IMPALA-13181: Disable tuple caching for locations with limits
Limits can be non-deterministic, because they can depend on
the order in which rows are processed. This marks locations with
limits ineligible for tuple caching. In future, this can be
refined the rule to allow caching in locations that are
deterministic (e.g. limits on sorted inputs).
Testing:
- Added test cases to TupleCacheTest
Change-Id: Iedcb73082325f1dfe397fbda07c2b7b4817efa5e
---
M fe/src/main/java/org/apache/impala/common/ThriftSerializationCtx.java
M fe/src/main/java/org/apache/impala/planner/PlanNode.java
M fe/src/main/java/org/apache/impala/planner/TupleCacheInfo.java
M fe/src/test/java/org/apache/impala/planner/TupleCacheTest.java
4 files changed, 35 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/21857/2
--
To view, visit http://gerrit.cloudera.org:8080/21857
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iedcb73082325f1dfe397fbda07c2b7b4817efa5e
Gerrit-Change-Number: 21857
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
