Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/11422

to look at the new patch set (#4).

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

[location_awareness] Add location info in ksck report

This patch adds additional tablet server location info
into the ksck report.

Before:
Tablet Server Summary
               UUID               |     Address     | Status
----------------------------------+-----------------+---------
 21b1960d8b164c869a3e600ed8bad1ea | 127.0.0.1:63000 | HEALTHY
 2c55bc50306a4463aa5a935aa97caff6 | 127.0.0.1:62973 | HEALTHY
 a8dee6038c494b97b883d98292f7771f | 127.0.0.1:62983 | HEALTHY
 d3664283918a434cbc5a8e067fa7c3b0 | 127.0.0.1:62978 | HEALTHY
...

After:
Tablet Server Summary
               UUID               |     Address     |       Location        | 
Status
----------------------------------+-----------------+-----------------------+---------
 21b1960d8b164c869a3e600ed8bad1ea | 127.0.0.1:63000 | /foo-bar0/BAAZ.9-quux | 
HEALTHY
 2c55bc50306a4463aa5a935aa97caff6 | 127.0.0.1:62973 | <none>                | 
HEALTHY
 a8dee6038c494b97b883d98292f7771f | 127.0.0.1:62983 | <none>                | 
HEALTHY
 d3664283918a434cbc5a8e067fa7c3b0 | 127.0.0.1:62978 | <none>                | 
HEALTHY

       Location        |  Count
-----------------------+---------
 <none>                |       3
 /foo-bar0/BAAZ.9-quux |       1
...

Change-Id: Ideff2dd4975c99a1135002624debbbb2620fb95c
---
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/tablet_server-internal.cc
M src/kudu/client/tablet_server-internal.h
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck.cc
M src/kudu/tools/ksck.h
M src/kudu/tools/ksck_remote-test.cc
M src/kudu/tools/ksck_remote.cc
M src/kudu/tools/ksck_remote.h
M src/kudu/tools/ksck_results.cc
M src/kudu/tools/ksck_results.h
12 files changed, 118 insertions(+), 30 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/11422/4
--
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: newpatchset
Gerrit-Change-Id: Ideff2dd4975c99a1135002624debbbb2620fb95c
Gerrit-Change-Number: 11422
Gerrit-PatchSet: 4
Gerrit-Owner: Fengling Wang <fw...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Fengling Wang <fw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>

Reply via email to