Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21857 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/21857 Reviewed-by: Michael Smith <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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, 41 insertions(+), 3 deletions(-) Approvals: Michael Smith: Looks good to me, approved Impala Public Jenkins: Verified -- 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: merged Gerrit-Change-Id: Iedcb73082325f1dfe397fbda07c2b7b4817efa5e Gerrit-Change-Number: 21857 Gerrit-PatchSet: 5 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Yida Wu <[email protected]>
