Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11017 )

Change subject: Add alter_external_catalogs flag to table rename tool
......................................................................


Patch Set 2: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11017/2/src/kudu/client/client.h
File src/kudu/client/client.h:

http://gerrit.cloudera.org:8080/#/c/11017/2/src/kudu/client/client.h@61
PS2, Line 61: class KuduClient;
nit: this can be moved to L71.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9f5d97f234283b7740df727e085523c7f0b09735
Gerrit-Change-Number: 11017
Gerrit-PatchSet: 2
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Tue, 24 Jul 2018 21:18:29 +0000
Gerrit-HasComments: Yes

Reply via email to