Revision 7168d16103cb9e48bde9f435c21b0f02186f5d7f authored by Levi Tamasi on 20 December 2019, 02:03:24 UTC, committed by Levi Tamasi on 20 December 2019, 02:29:39 UTC
Summary: BlobDB currently only supports using the default column family. The earlier code enforces this by comparing the `ColumnFamilyHandle` passed to the `Get`/`Put`/etc. call with the handle returned by `DefaultColumnFamily` (which, at the end of the day, comes from `DBImpl::default_cf_handle_`). Since other `ColumnFamilyHandle`s can also point to the default column family, this can reject legitimate requests as well. (As an example, with the earlier code, the handle returned by `BlobDB::Open` cannot actually be used in API calls.) The patch fixes this by comparing only the IDs of the column family handles instead of the pointers themselves. Pull Request resolved: https://github.com/facebook/rocksdb/pull/6226 Test Plan: `make check` Differential Revision: D19187461 Pulled By: ltamasi fbshipit-source-id: 54ce2e12ebb1f07e6d1e70e3b1e0213dfa94bda2
1 parent d848059
File | Mode | Size |
---|---|---|
backupable | ||
blob_db | ||
cassandra | ||
checkpoint | ||
compaction_filters | ||
convenience | ||
leveldb_options | ||
memory | ||
merge_operators | ||
option_change_migration | ||
options | ||
persistent_cache | ||
simulator_cache | ||
table_properties_collectors | ||
trace | ||
transactions | ||
ttl | ||
write_batch_with_index | ||
debug.cc | -rw-r--r-- | 2.6 KB |
env_librados.cc | -rw-r--r-- | 36.3 KB |
env_librados.md | -rw-r--r-- | 4.6 KB |
env_librados_test.cc | -rw-r--r-- | 34.6 KB |
env_mirror.cc | -rw-r--r-- | 7.9 KB |
env_mirror_test.cc | -rw-r--r-- | 7.1 KB |
env_timed.cc | -rw-r--r-- | 5.1 KB |
env_timed_test.cc | -rw-r--r-- | 1.2 KB |
merge_operators.h | -rw-r--r-- | 2.0 KB |
object_registry.cc | -rw-r--r-- | 2.8 KB |
object_registry_test.cc | -rw-r--r-- | 5.8 KB |
util_merge_operators_test.cc | -rw-r--r-- | 3.3 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...