Abhishek Chennaka has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22312 )

Change subject: [docs] Elaborate rack aware rebalancing
......................................................................


Patch Set 1:

(11 comments)

http://gerrit.cloudera.org:8080/#/c/22312/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22312/1//COMMIT_MSG@7
PS1, Line 7: rack aware
> nit: rack-aware
Done


http://gerrit.cloudera.org:8080/#/c/22312/1//COMMIT_MSG@9
PS1, Line 9: Adding some examples and a small note based on field feedback on 
the
> Consider separating the newly added examples into their own section, and pr
Done


http://gerrit.cloudera.org:8080/#/c/22312/1//COMMIT_MSG@10
PS1, Line 10: rack aware
> nit: rack-aware
Done


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

http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1577
PS1, Line 1577: For example
> nit: add a comma
Done


http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1578
PS1, Line 1578: (
> nit: add space before the opening parenthesis
Done


http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1578
PS1, Line 1578: 9 replicas
> nit: 9 tablet replicas total
Done


http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1581
PS1, Line 1581: Running
> running
Done


http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1585
PS1, Line 1585: |
> nit: same as above comment
Done


http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1585
PS1, Line 1585: |
> nit: align this column divider line with one above, same for rest of lines
Done


http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1604
PS1, Line 1604: For example
> nit: add a comma after "For example"
Done


http://gerrit.cloudera.org:8080/#/c/22312/1/docs/administration.adoc@1611
PS1, Line 1611: |
> nit: align this line with the ones below where applicable, won't leave a co
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6724c8cdd69167fabf51b66a462dfa25338c057d
Gerrit-Change-Number: 22312
Gerrit-PatchSet: 1
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Comment-Date: Thu, 09 Jan 2025 00:33:21 +0000
Gerrit-HasComments: Yes

Reply via email to