You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Dan Burkert (Code Review)" <ge...@cloudera.org> on 2016/03/21 19:31:29 UTC
[kudu-CR] [java-client] implement ColumnPredicate API
Hello David Ribeiro Alves, Jean-Daniel Cryans, Todd Lipcon,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/2591
to review the following change.
Change subject: [java-client] implement ColumnPredicate API
......................................................................
[java-client] implement ColumnPredicate API
This commit adds a new ColumnPredicate API which corresponds to the new
ColumnPredicatePB predicate type for scans. The ColumnPredicate allows
specifying exclusive upper bound predicates on columns. This commit is a first
step towards predicate optimization and partition pruning, but stops short of
actually implementing those optimizations.
This commit does not attempt to change any callers of the now deprecated
ColumnRangePredicate API.
Change-Id: Icdca28139a2f4f15633cfd872e372429bad831cd
---
M java/kudu-client/src/main/java/org/kududb/client/AbstractKuduScannerBuilder.java
M java/kudu-client/src/main/java/org/kududb/client/AsyncKuduScanner.java
A java/kudu-client/src/main/java/org/kududb/client/ColumnPredicate.java
M java/kudu-client/src/main/java/org/kududb/client/ColumnRangePredicate.java
M java/kudu-client/src/main/java/org/kududb/client/KuduScanner.java
M java/kudu-client/src/main/java/org/kududb/client/ProtobufHelper.java
A java/kudu-client/src/test/java/org/kududb/client/TestColumnPredicate.java
7 files changed, 1,206 insertions(+), 38 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/2591/1
--
To view, visit http://gerrit.cloudera.org:8080/2591
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icdca28139a2f4f15633cfd872e372429bad831cd
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>