Revision 9046bdc5d3a4e8774e08e4b12784d37c2b001c9f authored by anand76 on 21 August 2019, 17:21:41 UTC, committed by Facebook Github Bot on 21 August 2019, 17:23:23 UTC
Summary:
The batched MultiGet() implementation was not correctly handling bloom filter lookups when whole_key_filtering is disabled. It was incorrectly skipping keys not in the prefix_extractor domain, and not calling transform for keys in domain. This PR fixes both problems by moving the domain check and transformation to the FilterBlockReader.

Tests:
Unit test (confirmed failed before the fix)
make check
Pull Request resolved: https://github.com/facebook/rocksdb/pull/5665

Differential Revision: D16902380

Pulled By: anand1976

fbshipit-source-id: a6be81ad68a6e37134a65246aec7a2c590eccf00
1 parent 7bc18e2
History
File Mode Size
buckify_rocksdb.py -rw-r--r-- 7.4 KB
rocks_test_runner.sh -rwxr-xr-x 268 bytes
targets_builder.py -rw-r--r-- 2.3 KB
targets_cfg.py -rw-r--r-- 4.4 KB
util.py -rw-r--r-- 3.2 KB

back to top