Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13757 )
Change subject: [docs] update Hive Metastore integration and Impala integration docs ...................................................................... Patch Set 6: Code-Review+1 (3 comments) http://gerrit.cloudera.org:8080/#/c/13757/4/docs/hive_metastore.adoc File docs/hive_metastore.adoc: http://gerrit.cloudera.org:8080/#/c/13757/4/docs/hive_metastore.adoc@209 PS4, Line 209: : The `kudu hms precheck` tool sca > I just noted that we recommend dropping and recreating them. How's that? That looks good to me. Thank you! http://gerrit.cloudera.org:8080/#/c/13757/4/docs/hive_metastore.adoc@277 PS4, Line 277: + : [source,bash] : ---- > Correct me if I'm wrong, but i think this may be referring to tables with t It seems so. From the other side, if a table's name is Hive-compatible, then why to rename it at all? It seems it's better to make this thread as resolved since it's getting confusing. Probably, the right answer to my original question is 'go read the docs' :) http://gerrit.cloudera.org:8080/#/c/13757/4/docs/hive_metastore.adoc@314 PS4, Line 314: ---- : + : > I don't think it matters; external table creation is ignored by Kudu, no? If 'enabling' HMS integration means only configuring corresponding components (but not running the 'hms' tools), then the order doesn't matter, indeed. When I first read this piece I was under impression that 'enable' means the whole set of migration procedures, but, apparently, it's not. -- To view, visit http://gerrit.cloudera.org:8080/13757 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8726a39d56c4e9954f208700e99e7bcf2bbc290d Gerrit-Change-Number: 13757 Gerrit-PatchSet: 6 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Priyanka Chheda <pchh...@cloudera.com> Gerrit-Comment-Date: Wed, 03 Jul 2019 06:02:42 +0000 Gerrit-HasComments: Yes