Jean-Daniel Cryans has submitted this change and it was merged. Change subject: ksck: usability improvements in error messages ......................................................................
ksck: usability improvements in error messages - print the address of tablet servers that fail to connect - print the table names of bad tablets This doesn't lend itself to new test assertions, but I verified the changes both against a real cluster and by inspecting the output of 'ksck-test'. Change-Id: I3cd6a6e0f82d03f890ac70af32814277dfd33c54 Reviewed-on: http://gerrit.cloudera.org:8080/3225 Reviewed-by: Jean-Daniel Cryans Tested-by: Kudu Jenkins (cherry picked from commit 3ab902eb33bb6f0d7e5ac56892ddcf4d0f1f1300) Reviewed-on: http://gerrit.cloudera.org:8080/3226 --- 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.cc M src/kudu/tools/ksck_remote.h 5 files changed, 47 insertions(+), 31 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3226 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3cd6a6e0f82d03f890ac70af32814277dfd33c54 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: branch-0.9.x Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins
