Mike Percy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13356 )

Change subject: [backup] Add a basic CLI tool
......................................................................


Patch Set 3:

Looks good. Feature requests:
1. Make the list of tables optional, or provide another command to list all 
tables.
2. Provide the option for headerless TSV output that is easy to parse with a 
script, which would make it easy to write a quick test for this CLI tool.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib5a4d3b44c77a06fa1b5d2b09506ba5a6e1b52c1
Gerrit-Change-Number: 13356
Gerrit-PatchSet: 3
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Mon, 20 May 2019 21:00:34 +0000
Gerrit-HasComments: No

Reply via email to