Dan Burkert has submitted this change and it was merged.

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
Reviewed-on: http://gerrit.cloudera.org:8080/2591
Tested-by: Kudu Jenkins
Reviewed-by: Jean-Daniel Cryans
---
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
A java/kudu-client/src/test/java/org/kududb/client/TestScanPredicate.java
M java/kudu-client/src/test/resources/log4j.properties
13 files changed, 2,154 insertions(+), 62 deletions(-)

Approvals:
  Jean-Daniel Cryans: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icdca28139a2f4f15633cfd872e372429bad831cd
Gerrit-PatchSet: 10
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to