Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13410 )
Change subject: hms: followup to 6a8d1d2 ...................................................................... hms: followup to 6a8d1d2 Commit 6a8d1d2 left room for a little bit of refactoring and testing. This patch fills this room: - minor refactoring in tool_action_hms - tested that altering and dropping external tables won't affect underlying Kudu data There are no functional changes in this patch. Change-Id: I0d624258e1d177363fceba6b40d6aee77b142b54 Reviewed-on: http://gerrit.cloudera.org:8080/13410 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Andrew Wong <[email protected]> Reviewed-by: Hao Hao <[email protected]> --- M src/kudu/integration-tests/hms_itest-base.cc M src/kudu/integration-tests/hms_itest-base.h M src/kudu/integration-tests/master_hms-itest.cc M src/kudu/tools/tool_action_hms.cc 4 files changed, 66 insertions(+), 29 deletions(-) Approvals: Alexey Serbin: Looks good to me, but someone else must approve Andrew Wong: Verified Hao Hao: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13410 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0d624258e1d177363fceba6b40d6aee77b142b54 Gerrit-Change-Number: 13410 Gerrit-PatchSet: 6 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot (241)
