Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22667
Change subject: IMPALA-13886: Fix order of recursion for complex types for tuple caching ...................................................................... IMPALA-13886: Fix order of recursion for complex types for tuple caching Queries that use complex types fail with an IllegalStateException when running with tuple caching. The issue is that the logic for registering tuples is recursing in the wrong order. It processes the parent before moving on to the child, but the parent references tuple id's in the child and cannot be processed first. This switches the ordering so that the children are processed first. Testing: - Added a sanity check for complex types in TupleCacheTest Change-Id: I853c06e76422382c628850181e3c7e4fdf86efc2 --- M fe/src/main/java/org/apache/impala/planner/TupleCacheInfo.java M fe/src/test/java/org/apache/impala/planner/TupleCacheTest.java 2 files changed, 12 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/67/22667/1 -- To view, visit http://gerrit.cloudera.org:8080/22667 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I853c06e76422382c628850181e3c7e4fdf86efc2 Gerrit-Change-Number: 22667 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>
