Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11197 )
Change subject: Add modify_external_catalogs flag to table delete tool ...................................................................... Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/11197/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/11197/5//COMMIT_MSG@7 PS5, Line 7: modify_external_catalogs > modify_external_catalogs Done http://gerrit.cloudera.org:8080/#/c/11197/4/src/kudu/client/client.h File src/kudu/client/client.h: http://gerrit.cloudera.org:8080/#/c/11197/4/src/kudu/client/client.h@1233 PS4, Line 1233: FRIEND_TEST(kudu::MasterHmsUpgradeTest, TestRenameExistingTables); > I think this should be done as part of this patch, since the same lines of Done http://gerrit.cloudera.org:8080/#/c/11197/5/src/kudu/tools/tool_action_table.cc File src/kudu/tools/tool_action_table.cc: http://gerrit.cloudera.org:8080/#/c/11197/5/src/kudu/tools/tool_action_table.cc@43 PS5, Line 43: "when renaming or dropping a table."); > 'renaming' -> 'renaming or dropping' Done -- To view, visit http://gerrit.cloudera.org:8080/11197 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0a128fb53c974a5c839786204d56408681b434e8 Gerrit-Change-Number: 11197 Gerrit-PatchSet: 6 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Fri, 07 Sep 2018 04:14:07 +0000 Gerrit-HasComments: Yes
