Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11018 )
Change subject: hms-tool: refactor check tool and combine upgrade and fix ...................................................................... hms-tool: refactor check tool and combine upgrade and fix This commit combines the 'hms upgrade' and 'hms fix' tools. The tools previously had overlapping responsibilities for checking some types of inconsistencies. The 'fix' tool now has flags which can control whether it attempts certain types of fixes: --drop_orphan_hms_tables=false --create_missing_hms_tables=true --fix_inconsistent_tables=true --upgrade_hms_tables=true `drop_orphan_hms_tables` defaults to false since it deletes Hive tables, and the tool can not rule-out that they are being used. Additionally, a --dryrun flag is provided in order to print steps that would be taken without actually making modifications. The checking logic has been expanded to account for more potential inconsistencies, and where possible the fix tool now can automatically repair these inconsistencies. Finally, the input prompt and default database functionality has been removed in order to simplify the tool. Instead, the check tool will issue hints including instructions for how to rename tables without a database or with a Hive-incompatible name using the 'kudu table rename_table' tool. We can always add this functionality back in the future if we determine it helps out users. Change-Id: Ieee478002eb56a278cfd74255670baaf28109b8f Reviewed-on: http://gerrit.cloudera.org:8080/11018 Tested-by: Dan Burkert <[email protected]> Reviewed-by: Hao Hao <[email protected]> --- M src/kudu/gutil/strings/escaping.h M src/kudu/hms/hms_catalog.cc M src/kudu/hms/hms_catalog.h M src/kudu/tools/CMakeLists.txt M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_hms.cc 6 files changed, 887 insertions(+), 818 deletions(-) Approvals: Dan Burkert: Verified Hao Hao: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11018 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ieee478002eb56a278cfd74255670baaf28109b8f Gerrit-Change-Number: 11018 Gerrit-PatchSet: 12 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-Reviewer: Tidy Bot
