Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/12969 )
Change subject: KUDU-2449: "Tablet server addition" best practices ...................................................................... Patch Set 3: (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 like: Instead, they will only be utilized when new tablets are created or existing tablets need to be replicated. 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: To add additional tablet servers.. 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_addrs) 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 running ksck? http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@824 PS3, Line 824: checked in Maybe "healthy" in place of checked in. -- 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: 3 Gerrit-Owner: Mitch Barnett <mbarn...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mitch Barnett <mbarn...@cloudera.com> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-Comment-Date: Tue, 09 Apr 2019 22:04:23 +0000 Gerrit-HasComments: Yes
