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/26 02:18:24 UTC
[kudu-CR] [java-client] implement KuduPredicate API
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2591
to look at the new patch set (#5).
Change subject: [java-client] implement KuduPredicate API
......................................................................
[java-client] implement KuduPredicate API
This commit adds a new class KuduPredicate, which corresponds to the new
ColumnPredicatePB predicate type for scans. The KuduPredicate allows specifying
exclusive upper bound and lower 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
M java/kudu-client/src/main/java/org/kududb/client/Bytes.java
M java/kudu-client/src/main/java/org/kududb/client/ColumnRangePredicate.java
A java/kudu-client/src/main/java/org/kududb/client/KuduPredicate.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
M java/kudu-client/src/test/java/org/kududb/client/BaseKuduTest.java
M java/kudu-client/src/test/java/org/kududb/client/TestBytes.java
M java/kudu-client/src/test/java/org/kududb/client/TestKuduClient.java
A java/kudu-client/src/test/java/org/kududb/client/TestKuduPredicate.java
M java/kudu-client/src/test/resources/log4j.properties
12 files changed, 1,428 insertions(+), 62 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/91/2591/5
--
To view, visit http://gerrit.cloudera.org:8080/2591
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icdca28139a2f4f15633cfd872e372429bad831cd
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>