Skip to content

Commit

Permalink
[client] Add a note that only FAST_HASH is supported in Bloom filter …
Browse files Browse the repository at this point in the history
…predicate

Add explicit note that only FAST_HASH is supported in the Bloom filter
predicate in client API.

There is already validation in BlockBloomFilter for hash algorithm.

Test:
- Verified the generated HTML output.

Change-Id: I5c683fe3be3e9ad2168562c5940e500fffac5316
Reviewed-on: http://gerrit.cloudera.org:8080/15608
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
  • Loading branch information
bbhavsar authored and adembo committed Mar 31, 2020
1 parent 263c3aa commit 03d79bd
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/kudu/client/scan_predicate.h
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,7 @@ class KUDU_EXPORT KuduBloomFilterBuilder {
/// @param [in] hash_algorithm
/// Hash algorithm used to hash keys before inserting to the Bloom filter.
/// If not provided, defaults to FAST_HASH.
/// @note Currently only FAST_HASH is supported.
/// @return Reference to the updated object.
KuduBloomFilterBuilder& hash_algorithm(HashAlgorithm hash_algorithm);

Expand Down

0 comments on commit 03d79bd

Please sign in to comment.