Will Berkeley has submitted this change and it was merged. Change subject: [tools] Use PrintTable to format ksck's consensus matrix ......................................................................
[tools] Use PrintTable to format ksck's consensus matrix The first version of ksck's consensus matrix used its own table code, but actually there was already nice table code available in tool_action_common.h. This switches ksck to use it. It also generalizes the table code to output to a generic ostream instead of only cout; this was necessary for ksck-test but might be useful in other ways later. Change-Id: I8d77005f20091e6778702580e3a269d9689c5b0a Reviewed-on: http://gerrit.cloudera.org:8080/7043 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <[email protected]> Reviewed-by: Mike Percy <[email protected]> --- M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck.cc M src/kudu/tools/tool_action_common.cc M src/kudu/tools/tool_action_common.h M src/kudu/tools/tool_action_master.cc M src/kudu/tools/tool_action_tserver.cc 6 files changed, 107 insertions(+), 140 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/7043 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d77005f20091e6778702580e3a269d9689c5b0a Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
