You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Anonymous Coward (Code Review)" <ge...@cloudera.org> on 2018/01/04 23:53:23 UTC
[kudu-CR] Added steps to update HMS after migrating to multiple Kudu masters.
arodoni@cloudera.com has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8946
Change subject: Added steps to update HMS after migrating to multiple Kudu masters.
......................................................................
Added steps to update HMS after migrating to multiple Kudu masters.
Change-Id: I920930f3e21c69effb8a5afc9c05ba9eb9e5285d
---
M docs/administration.adoc
1 file changed, 23 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/8946/1
--
To view, visit http://gerrit.cloudera.org:8080/8946
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I920930f3e21c69effb8a5afc9c05ba9eb9e5285d
Gerrit-Change-Number: 8946
Gerrit-PatchSet: 1
Gerrit-Owner: arodoni@cloudera.com
[kudu-CR] Added steps to update HMS after migrating to multiple Kudu masters.
Posted by "Thomas Tauber-Marshall (Code Review)" <ge...@cloudera.org>.
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8946 )
Change subject: Added steps to update HMS after migrating to multiple Kudu masters.
......................................................................
Patch Set 1:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/8946/1/docs/administration.adoc
File docs/administration.adoc:
http://gerrit.cloudera.org:8080/#/c/8946/1/docs/administration.adoc@382
PS1, Line 382: master-1
Feels a little weird to use the same values as the DNS aliases in the section specifically for if you didn't set the aliases up.
Maybe we can make it clear that these are host names by making them something like 'master-1.example.com' etc. or if that's too verbose then maybe just 'host-1' etc.
http://gerrit.cloudera.org:8080/#/c/8946/1/docs/administration.adoc@383
PS1, Line 383: PARAM_VALUE = 'old-master';
How about we add "AND PARAM_KEY = 'kudu.master_addresses'" here.
In practice, I think this is unlikely to make a difference, but better to be safe.
--
To view, visit http://gerrit.cloudera.org:8080/8946
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I920930f3e21c69effb8a5afc9c05ba9eb9e5285d
Gerrit-Change-Number: 8946
Gerrit-PatchSet: 1
Gerrit-Owner: arodoni@cloudera.com
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall <tm...@cloudera.com>
Gerrit-Comment-Date: Fri, 05 Jan 2018 00:41:10 +0000
Gerrit-HasComments: Yes
[kudu-CR] Added steps to update HMS after migrating to multiple Kudu masters.
Posted by "Anonymous Coward (Code Review)" <ge...@cloudera.org>.
arodoni@cloudera.com has abandoned this change. ( http://gerrit.cloudera.org:8080/8946 )
Change subject: Added steps to update HMS after migrating to multiple Kudu masters.
......................................................................
Abandoned
--
To view, visit http://gerrit.cloudera.org:8080/8946
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: abandon
Gerrit-Change-Id: I920930f3e21c69effb8a5afc9c05ba9eb9e5285d
Gerrit-Change-Number: 8946
Gerrit-PatchSet: 1
Gerrit-Owner: arodoni@cloudera.com
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Thomas Tauber-Marshall <tm...@cloudera.com>