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

Change subject: IMPALA-10435: Extend 'compute incremental stats' syntax to 
support a list of columns
......................................................................


Patch Set 3:

(1 comment)

Thanks, this makes sense I think. Can you test one more edge case for me?

http://gerrit.cloudera.org:8080/#/c/16947/3/testdata/workloads/functional-query/queries/QueryTest/compute-stats-incremental.test
File 
testdata/workloads/functional-query/queries/QueryTest/compute-stats-incremental.test:

http://gerrit.cloudera.org:8080/#/c/16947/3/testdata/workloads/functional-query/queries/QueryTest/compute-stats-incremental.test@830
PS3, Line 830: ====
Can you also run compute incremental stats without specifying a partition to 
see which partitions it recomputes stats on?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4dcc2d4458679c39581446f6d87bb7903803f09b
Gerrit-Change-Number: 16947
Gerrit-PatchSet: 3
Gerrit-Owner: liuyao <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: Bikramjeet Vig <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
Gerrit-Reviewer: liuyao <[email protected]>
Gerrit-Comment-Date: Thu, 21 Jan 2021 00:52:16 +0000
Gerrit-HasComments: Yes

Reply via email to