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
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 |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...