Hello Adar Dembo, Will Berkeley,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/7259

to review the following change.

Change subject: tools: refactor table-printing code a bit
......................................................................

tools: refactor table-printing code a bit

This changes the table-printing code to use a class and add some extra
safety checks that the added columns have matching numbers of rows.

Change-Id: I7e9556a14b6745f6bd07d1aee111bfa5cd9297fe
---
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
5 files changed, 76 insertions(+), 28 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/59/7259/1
-- 
To view, visit http://gerrit.cloudera.org:8080/7259
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7e9556a14b6745f6bd07d1aee111bfa5cd9297fe
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>

Reply via email to