Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15247 )
Change subject: [client] Remove range bounds in Bloom filter predicate ...................................................................... [client] Remove range bounds in Bloom filter predicate Lower and upper range bounds are required to allow merging a range predicate with Bloom filter predicate on the same column. Since the merging is done on the client side, the range bounds are required in the wire protocol but we don't need them in the public client APIs as client allows combining multiple predicates. Change-Id: I60fa19e69ececa97e96eb6ed5f082406b0aab890 Reviewed-on: http://gerrit.cloudera.org:8080/15247 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/client/predicate-test.cc M src/kudu/client/scan_predicate-internal.h M src/kudu/client/scan_predicate.cc M src/kudu/common/column_predicate.h M src/kudu/common/common.proto 7 files changed, 40 insertions(+), 73 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15247 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I60fa19e69ececa97e96eb6ed5f082406b0aab890 Gerrit-Change-Number: 15247 Gerrit-PatchSet: 2 Gerrit-Owner: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yao Xu <[email protected]> Gerrit-Reviewer: helifu <[email protected]>
