Mitch Barnett has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12969 )

Change subject: KUDU-2450: "Tablet server addition" best practices
......................................................................


Patch Set 4:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc
File docs/administration.adoc:

http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@808
PS3, Line 808: - only when new tablets are created or
             : existing tablets need to be replicated will the newly added 
tablet servers be
             : targeted for replica placement, which can lead to imbalanced 
nodes.
> Nit: I had a hard time following this. Maybe break it into two sentences li
Done


http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@812
PS3, Line 812: When adding additional tablet servers to an already existing 
cluster, the
> Maybe start with:
Done


http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@817
PS3, Line 817: correctly configured to point
             : to the pre-existing cluster
> Maybe elaborate by linking/showing the config needed. (--tserver_master_add
Done


http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@820
PS3, Line 820:  A quick method for veryifying they've successfully checked in
             : with the existing Master instances is to view the Kudu Master 
WebUI,
             : specifically the '/tablet-servers' section, and validate that 
the newly
             : added instances are registered, and heartbeating.
> Is there a tool method that would be work and be more concrete? Maybe runni
'ksck' would show the same information in a less readable format, imho. I think 
checking the WebUI is the most definitive way, as it shows the current tServers 
that are alive/dead and when their last heartbeat was. 'ksck' doesn't give the 
same kind of feedback, outside of "scanned ## of tServers".

I can change it if you think it'd be better suited to 'ksck', but I think the 
WebUI is the better option.


http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@824
PS3, Line 824: checked in
> Maybe "healthy" in place of checked in.
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7e44b4a84a8ca8048b8dc930cc1ee228daef6cf9
Gerrit-Change-Number: 12969
Gerrit-PatchSet: 4
Gerrit-Owner: Mitch Barnett <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mitch Barnett <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Tue, 09 Apr 2019 22:24:55 +0000
Gerrit-HasComments: Yes

Reply via email to