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 9: (4 comments) http://gerrit.cloudera.org:8080/#/c/13757/9/docs/hive_metastore.adoc File docs/hive_metastore.adoc: http://gerrit.cloudera.org:8080/#/c/13757/9/docs/hive_metastore.adoc@293 PS9, Line 293: in Impala for impalad ? It seems they also have the same run-time flag for both Catalog Server and StateStore, but I guess it's just some implementation artifact and the effective value is taken from the bespoken run-time flag of the impalad process. http://gerrit.cloudera.org:8080/#/c/13757/9/docs/hive_metastore.adoc@294 PS9, Line 294: host ports maybe, replace the implicitly introduced 'host port' entity with 'RPC endpoint'? I.e., The tools compare the value of the `kudu.master_addresses` table property ... against the RPC endpoints of Kudu master(s). http://gerrit.cloudera.org:8080/#/c/13757/9/docs/hive_metastore.adoc@295 PS9, Line 295: address values RPC endpoints ? http://gerrit.cloudera.org:8080/#/c/13757/9/docs/hive_metastore.adoc@296 PS9, Line 296: an argument to the : tools an extra argument for 'hms check' and 'hms fix' tools. As far as I can see, only 'hms check' and 'hms fix' tools have the --ignore_other_clusters run-time flag. -- 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: 9 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Alex Rodoni <[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: Priyanka Chheda <[email protected]> Gerrit-Comment-Date: Wed, 03 Jul 2019 22:10:32 +0000 Gerrit-HasComments: Yes
