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 (#6). 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 the 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"]' All sub-predicates combined together as '[operator, sub-predicate1, sub-predicate2, ...]', and only 'AND' operator is supported now. e.g. '["AND", [">=", "col1", "value"], ["NOTNULL", "col2"]]') Change-Id: Ieac340b70a9eaf131f82a2b7d61336211d1d48f8 --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_table.cc 2 files changed, 568 insertions(+), 18 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/12167/6 -- 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: 6 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>