Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10303 )
Change subject: KUDU-2191: HMS inconsistent metadata fix tool ...................................................................... Patch Set 9: (2 comments) http://gerrit.cloudera.org:8080/#/c/10303/8/src/kudu/tools/tool_action_hms.cc File src/kudu/tools/tool_action_hms.cc: http://gerrit.cloudera.org:8080/#/c/10303/8/src/kudu/tools/tool_action_hms.cc@483 PS8, Line 483: cout << Substitute("Renaming Kudu table $0 [id=$1] to $2 to match the Hive " : "Metastore catalog.", kudu_table_name, table_id, hms_table_name) : << endl; > Are these lines redundant with the 'Renaming...' message? Done http://gerrit.cloudera.org:8080/#/c/10303/8/src/kudu/tools/tool_action_hms.cc@486 PS8, Line 486: ent, > add a space between the table name and opening brace here and below. This Done -- To view, visit http://gerrit.cloudera.org:8080/10303 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I63c694b5d9877cfbd218139f2e9ecf05fd69c997 Gerrit-Change-Number: 10303 Gerrit-PatchSet: 9 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Tue, 26 Jun 2018 21:24:23 +0000 Gerrit-HasComments: Yes
