Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16140 )

Change subject: IMPALA-9917: grouping() and grouping_id() support
......................................................................


Patch Set 18: Verified+1 Code-Review+2

This just failed because of a merge marker that I failed to remove in the final 
line of a planner test. I removed that and reran the single affected test 
locally. I'm going to mark as verified to avoid the need to rerun.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If0b1640d606256c0fe9204d2a21a8f6d06abcdb6
Gerrit-Change-Number: 16140
Gerrit-PatchSet: 18
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: David Rorke <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Shant Hovsepian <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Tue, 14 Jul 2020 03:12:54 +0000
Gerrit-HasComments: No

Reply via email to