Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12969 )
Change subject: KUDU-2449: "Tablet server addition" best practices ...................................................................... Patch Set 3: (7 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@803 PS3, Line 803: Cluster no need for capitalization here. http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@810 PS3, Line 810: which can lead to imbalanced nodes. As written, it sounds like this is a cause of the actions listed just before, but I think what you're trying to say is that until these actions happen, the nodes will be imbalanced. Could you rewrite the sentence to clarify that? http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@812 PS3, Line 812: already Drop 'already'; 'existing' is sufficient. http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@822 PS3, Line 822: '/tablet-servers' We seem to be inconsistent as to whether we refer to webpages as /foo or `/foo` (this will syntax highlight them slightly). But I don't see anywhere that we do '/foo' (with single quotes). Could you change this (and 'ksck' below) to either /tablet-servers or `/tablet-servers`? http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@826 PS3, Line 826: Kudu Rebalancer Should probably refer to this as the "tablet rebalancing tool" or "rebalancing tool" as that's how it's called in the page behind the link. http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@827 PS3, Line 827: already Drop 'already' http://gerrit.cloudera.org:8080/#/c/12969/3/docs/administration.adoc@829 PS3, Line 829: it's its -- 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 <[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:14:28 +0000 Gerrit-HasComments: Yes
