Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11058 )

Change subject: [docs] Add changing master hostnames workflow
......................................................................


Patch Set 2:

(10 comments)

http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@663
PS1, Line 663: masters
> new masters ?
well, they're the same masters as previously, just the hostnames change, so I 
think it's fine


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@664
PS1, Line 664: resolve them.
> nit: missing period
Done


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@670
PS1, Line 670: 00000000000
> For this scenario, it's supposed to be 00000000000000000000000000000000 alw
Done


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@676
PS1, Line 676: new-master-name-1
> Does it make sense to point that those names might be either FQDN or IP add
it should always be FQDN, with IPs Kerberos wouldn't work properly by default 
(unless the hosts don't have hostnames at all, which is unlikely)


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@679
PS1, Line 679: master_addresses
> Does it make sense to mention that the '--rpc_bind_addresses' and '--webser
hostname change shouldn't affect that, unless they switch IP address as well


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@681
PS1, Line 681: tserver_master_addrs
> Does it make sense to mention that the '--rpc_bind_addresses' and '--webser
same as above


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@684
PS1, Line 684: . Start up the masters.
> Does it make sense to run some verification after this step?  Like 'kudu ma
verification is suggested in a later step


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@686
PS1, Line 686: Start all of the tablet servers.
> Does it make sense to run a verification step after that?  Say, 'sudo -u ku
verification is suggested in a later step


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@696
PS1, Line 696:
> Maybe, make those match with the names used for Raft configuration change i
Done


http://gerrit.cloudera.org:8080/#/c/11058/1/docs/administration.adoc@709
PS1, Line 709: ----
             : +
             : .. In `impala-shell`, run:
             : +
             : [source,bash]
             : ----
             : INVALIDATE METADATA;
             : ----
             : +
             : .. Verify updating the metadata worked
> I think it make sense to do so prior to Impala and HMS-related changes.
Done



--
To view, visit http://gerrit.cloudera.org:8080/11058
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0df87d5e294d8b7bf5c7b8f94a63599ffd7ebe03
Gerrit-Change-Number: 11058
Gerrit-PatchSet: 2
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Fri, 27 Jul 2018 09:09:46 +0000
Gerrit-HasComments: Yes

Reply via email to