Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14934 )
Change subject: [util] Import FastHash hash function to util ...................................................................... [util] Import FastHash hash function to util FastHash is a simple, robust, and efficient general purpose hash function from Google. It's fast, small in size and with least/no quality problems as per smhasher and hence a good candidate for BlockBloomFilter. C++ implementation has been ported straight from https://code.google.com/archive/p/fast-hash/ with minor style modifications. Java implementation has been adapted from the C implementation. Change-Id: Id0a21d6af10d9ba0dbd9ab46d73552d42976e8d7 Reviewed-on: http://gerrit.cloudera.org:8080/14934 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- A java/kudu-client/src/main/java/org/apache/kudu/util/HashUtil.java A java/kudu-client/src/test/java/org/apache/kudu/util/TestFashHash.java M src/kudu/util/hash_util-test.cc M src/kudu/util/hash_util.h 4 files changed, 260 insertions(+), 3 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14934 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id0a21d6af10d9ba0dbd9ab46d73552d42976e8d7 Gerrit-Change-Number: 14934 Gerrit-PatchSet: 8 Gerrit-Owner: Bankim Bhavsar <ban...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Bankim Bhavsar <ban...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
