Todd Lipcon 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
---
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/3225
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I3cd6a6e0f82d03f890ac70af32814277dfd33c54
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to