Todd Lipcon has posted comments on this change.

Change subject: ksck: colorize and clean up output
......................................................................


Patch Set 3:

> One thing I noticed is that you have a newline separating tablet output, but 
> not tables.

added

> It would be really helpful to print the time that TABLET_DATA_COPYING 
> operations started.  Not sure if this is exposed at all, maybe it's outside 
> the scope of ksck.

yea, that would have to be a change in the tablet_copy_client code which 
reports status back to the tablet peer

-- 
To view, visit http://gerrit.cloudera.org:8080/4129
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icc5196d63cbcbcbb2a9aba1ff17377b678c104bd
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to