Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11422 )

Change subject: [location_awareness] Add location info in ksck report
......................................................................


Patch Set 1:

(1 comment)

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

http://gerrit.cloudera.org:8080/#/c/11422/1//COMMIT_MSG@29
PS1, Line 29:         Tablet Server UUID        |       Location
            : ----------------------------------+-----------------------
            :  fc18b4bdb2ae4dd0a5717e8fe1f1de69 | <none>
            :  0f25d1891fce48789e06fdec493fb90e | <none>
            :  ad868d782dc743369d96a9e958811f81 | <none>
            :  c541b6ebc3a14f0f9bbee26d52545683 | /foo-bar0/BAAZ.9-quux
            :
            :        Location        |  Count
            : -----------------------+---------
            :  /foo-bar0/BAAZ.9-quux |       1
            :  <none>                |       3
> ksck output is already pretty lengthy so I don't want to introduce an extra
+1

Woops, indeed -- we have 'Tablet Server Summary' section.  I think the best fit 
for the tablet server location information is in that table.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ideff2dd4975c99a1135002624debbbb2620fb95c
Gerrit-Change-Number: 11422
Gerrit-PatchSet: 1
Gerrit-Owner: Fengling Wang <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Fengling Wang <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Wed, 12 Sep 2018 17:35:47 +0000
Gerrit-HasComments: Yes

Reply via email to