Hello Will Berkeley, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12167 to look at the new patch set (#16). Change subject: [tools] Add table scan tool ...................................................................... [tools] Add table scan tool This commit adds a basic tool to scan rows from a table. Several predicates can specified on the query. Unlike traditional SQL syntax, the scan tool's simple query predicates are represented in a simple JSON syntax. Three types of predicates are supported, including 'Comparison', 'InList' and 'IsNull'. * The 'Comparison' type support <=, <, ==, > and >=, which can be represented as '[operator, column_name, value]', e.g. '[">=", "col1", "value"]' * The 'InList' type can be represented as '["IN", column_name, [value1, value2, ...]]' e.g. '["IN", "col2", ["value1", "value2"]]' * The 'IsNull' type determine whether the value is NULL or not, which can be represented as '[operator, column_name]' e.g. '["NULL", "col1"]', or '["NOTNULL", "col2"]' Predicates can be combined together with predicate operators using the syntax [operator, predicate, predicate, ..., predicate]. For example, ["AND", [">=", "col1", "value"], ["NOTNULL", "col2"]] The only supported predicate operator is `AND`. Change-Id: Ieac340b70a9eaf131f82a2b7d61336211d1d48f8 --- M src/kudu/client/scanner-internal.h M src/kudu/tools/CMakeLists.txt M src/kudu/tools/kudu-tool-test.cc A src/kudu/tools/table_scanner.cc A src/kudu/tools/table_scanner.h M src/kudu/tools/tool_action_cluster.cc M src/kudu/tools/tool_action_common.cc M src/kudu/tools/tool_action_perf.cc M src/kudu/tools/tool_action_table.cc 9 files changed, 695 insertions(+), 32 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/12167/16 -- 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: newpatchset Gerrit-Change-Id: Ieac340b70a9eaf131f82a2b7d61336211d1d48f8 Gerrit-Change-Number: 12167 Gerrit-PatchSet: 16 Gerrit-Owner: Yingchun Lai <405403...@qq.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Yingchun Lai <405403...@qq.com>