Will Berkeley has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11422 )

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     | Status  | Location
----------------------------------+-----------------+---------+----------
 21b1960d8b164c869a3e600ed8bad1ea | 127.0.0.1:63000 | HEALTHY | /foo
 2c55bc50306a4463aa5a935aa97caff6 | 127.0.0.1:62973 | HEALTHY | <none>
 a8dee6038c494b97b883d98292f7771f | 127.0.0.1:62983 | HEALTHY | <none>
 d3664283918a434cbc5a8e067fa7c3b0 | 127.0.0.1:62978 | HEALTHY | <none>

Tablet Server Location Summary
 Location |  Count
----------+---------
 <none>   |       3
 /foo     |       1
...

Change-Id: Ideff2dd4975c99a1135002624debbbb2620fb95c
Reviewed-on: http://gerrit.cloudera.org:8080/11422
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <aser...@cloudera.com>
Reviewed-by: Will Berkeley <wdberke...@gmail.com>
---
M src/kudu/tools/CMakeLists.txt
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/tool.proto
9 files changed, 147 insertions(+), 28 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, but someone else must approve
  Will Berkeley: Looks good to me, approved

--
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: merged
Gerrit-Change-Id: Ideff2dd4975c99a1135002624debbbb2620fb95c
Gerrit-Change-Number: 11422
Gerrit-PatchSet: 16
Gerrit-Owner: Fengling Wang <fw...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Fengling Wang <fw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>

Reply via email to