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>