Yingchun Lai has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12167 )

Change subject: [tools] Add table scan tool
......................................................................


Patch Set 4:

> (5 comments)
 >
 > I left a few high-level comments, but at the top level: what is the
 > goal of this tool? Is it to serve as a barebones shell, like
 > impala-shell or MySQL shell? Is it meant for benchmarking scan
 > performance?
Yes, I think it's necessary to introduce a tool to benchmark the scan 
performance directly access the kudu server, but not the impala-shell or 
spark-shell, they may produce some extra time cost. And the scan tool build 
with in the kudu tool is some more convenient to use.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ieac340b70a9eaf131f82a2b7d61336211d1d48f8
Gerrit-Change-Number: 12167
Gerrit-PatchSet: 4
Gerrit-Owner: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Comment-Date: Sat, 12 Jan 2019 09:32:51 +0000
Gerrit-HasComments: No

Reply via email to