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

Change subject: IMPALA-7604: part 1: tests for agg cardinality
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14131/3/testdata/workloads/functional-planner/queries/PlannerTest/card-agg.test
File testdata/workloads/functional-planner/queries/PlannerTest/card-agg.test:

http://gerrit.cloudera.org:8080/#/c/14131/3/testdata/workloads/functional-planner/queries/PlannerTest/card-agg.test@397
PS3, Line 397: # Test overflow handling for single aggregation class.
> nit: add
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I59eaddbc5be253793293af064bb2d28a425564e1
Gerrit-Change-Number: 14131
Gerrit-PatchSet: 3
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Bikramjeet Vig <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Tue, 27 Aug 2019 23:53:40 +0000
Gerrit-HasComments: Yes

Reply via email to