Disable new Bloom filter assertion (#6128)

Summary:
A longstanding bug in our C interface can trigger this
assertion; see issue https://github.com/facebook/rocksdb/issues/6129. Disabling the assertion for now
(for 6.6.0) and will re-enable on fix of that bug.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/6128

Differential Revision: D18854899

Pulled By: pdillinger

fbshipit-source-id: 9eb5294b9f11b208dc1a8cc148aaa31e47ff892b
main
Peter Dillinger 5 years ago committed by Facebook Github Bot
parent ad528fe5ca
commit 6db57bc37f
  1. 3
      table/block_based/filter_policy.cc

@ -443,7 +443,8 @@ void BloomFilterPolicy::CreateFilter(const Slice* keys, int n,
std::string* dst) const { std::string* dst) const {
// We should ideally only be using this deprecated interface for // We should ideally only be using this deprecated interface for
// appropriately constructed BloomFilterPolicy // appropriately constructed BloomFilterPolicy
assert(mode_ == kDeprecatedBlock); // FIXME disabled because of bug in C interface; see issue #6129
//assert(mode_ == kDeprecatedBlock);
// Compute bloom filter size (in both bits and bytes) // Compute bloom filter size (in both bits and bytes)
uint32_t bits = static_cast<uint32_t>(n * whole_bits_per_key_); uint32_t bits = static_cast<uint32_t>(n * whole_bits_per_key_);

Loading…
Cancel
Save