Alex Behm has posted comments on this change.

Change subject: IMPALA-3938: Disallow implicit references between nested 
collections.
......................................................................


Patch Set 5: Code-Review+2

Thanks for replacing it. To clarify, my version had 100+ fewer LOC than your 
version and the whole test code fits into one screen, so imo, is easier to read.

-- 
To view, visit http://gerrit.cloudera.org:8080/4079
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I703c7e9c8c9c95c9fee714767367373eec10cd0c
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Christopher Channing <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Christopher Channing <[email protected]>
Gerrit-HasComments: No

Reply via email to