Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10054 )

Change subject: [tools] minor enhancements on 'kudu cluster ksck' output
......................................................................


Patch Set 5:

(1 comment)

> (1 comment)
 >
 > How does this behave with table and/or tablet filters? Can you
 > check what it does make sense and add a couple tests for it?

Yes, it works as expected -- the summary displays the counters related to the 
entities that satisfy table/tablet filters.  I added corresponding tests as 
well.

http://gerrit.cloudera.org:8080/#/c/10054/5//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/10054/5//COMMIT_MSG@27
PS5, Line 27: Masters        | 1
> nit: Can we order it
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0888ef640b215e1e0cf1f872f02cfe34f4ef5ba6
Gerrit-Change-Number: 10054
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Tue, 17 Apr 2018 21:05:27 +0000
Gerrit-HasComments: Yes

Reply via email to