Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15122 )
Change subject: [client] KUDU-2483 Add IN Bloom filter predicate to C++ client ...................................................................... [client] KUDU-2483 Add IN Bloom filter predicate to C++ client Instead of exposing BlockBloomFilter directly to clients, added a separate KuduBloomFilter class that uses PIMPL pattern to access the underlying BlockBloomFilter and allocator. To enable extending KuduBloomFilter in future added a builder class KuduBloomFilterBuilder. Change-Id: I4aa235a4c933ebce0bf3ec7fcb135098eccc4ea4 Reviewed-on: http://gerrit.cloudera.org:8080/15122 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/client/CMakeLists.txt M src/kudu/client/client.cc M src/kudu/client/client.h A src/kudu/client/hash-internal.h A src/kudu/client/hash.cc A src/kudu/client/hash.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/client/scan_predicate.h M src/kudu/client/value.h 11 files changed, 679 insertions(+), 11 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/15122 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4aa235a4c933ebce0bf3ec7fcb135098eccc4ea4 Gerrit-Change-Number: 15122 Gerrit-PatchSet: 9 Gerrit-Owner: Bankim Bhavsar <ban...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Bankim Bhavsar <ban...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yao Xu <ocla...@gmail.com> Gerrit-Reviewer: ZhangYao <triplesheep0...@gmail.com> Gerrit-Reviewer: helifu <hzhel...@corp.netease.com>