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

Change subject: IMPALA-4364: Query option to refresh updated partitions
......................................................................


Patch Set 1:

(1 comment)

I think this makes sense at a high level.

Are you thinking that we'd want to optimise this further before changing the 
default?

http://gerrit.cloudera.org:8080/#/c/16308/1/common/thrift/CatalogService.thrift
File common/thrift/CatalogService.thrift:

http://gerrit.cloudera.org:8080/#/c/16308/1/common/thrift/CatalogService.thrift@255
PS1, Line 255: refresh_modified_partitions
nit: name here doesn't match the query option name



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I50e8680509f4eb0712e7bb3de44df5f2952179af
Gerrit-Change-Number: 16308
Gerrit-PatchSet: 1
Gerrit-Owner: Vihang Karajgaonkar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Mon, 10 Aug 2020 18:09:08 +0000
Gerrit-HasComments: Yes

Reply via email to