BlobDB: Remove GC interval option (#5044)

Summary:
Remove BlobDBOptions.garbage_collection_interval_secs for now, since
garbage collection is not yet implemented in BlobDB.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/5044

Differential Revision: D14354046

Pulled By: ltamasi

fbshipit-source-id: 2b966b6d1e088ba9462f3ea73e115013562fbc04
main
Levi Tamasi 6 years ago committed by Facebook Github Bot
parent 04a2631dbe
commit 79b6ab43ce
  1. 3
      utilities/blob_db/blob_db.cc
  2. 3
      utilities/blob_db/blob_db.h
  3. 3
      utilities/blob_db/blob_db_impl.cc

@ -93,9 +93,6 @@ void BlobDBOptions::Dump(Logger* log) const {
ROCKS_LOG_HEADER( ROCKS_LOG_HEADER(
log, " BlobDBOptions.enable_garbage_collection: %d", log, " BlobDBOptions.enable_garbage_collection: %d",
enable_garbage_collection); enable_garbage_collection);
ROCKS_LOG_HEADER(
log, " BlobDBOptions.garbage_collection_interval_secs: %" PRIu64,
garbage_collection_interval_secs);
ROCKS_LOG_HEADER( ROCKS_LOG_HEADER(
log, " BlobDBOptions.disable_background_tasks: %d", log, " BlobDBOptions.disable_background_tasks: %d",
disable_background_tasks); disable_background_tasks);

@ -73,9 +73,6 @@ struct BlobDBOptions {
// blob files will be cleanup based on TTL. // blob files will be cleanup based on TTL.
bool enable_garbage_collection = false; bool enable_garbage_collection = false;
// Time interval to trigger garbage collection, in seconds.
uint64_t garbage_collection_interval_secs = 60;
// Disable all background job. Used for test only. // Disable all background job. Used for test only.
bool disable_background_tasks = false; bool disable_background_tasks = false;

@ -198,9 +198,6 @@ void BlobDBImpl::StartBackgroundTasks() {
tqueue_.add( tqueue_.add(
kReclaimOpenFilesPeriodMillisecs, kReclaimOpenFilesPeriodMillisecs,
std::bind(&BlobDBImpl::ReclaimOpenFiles, this, std::placeholders::_1)); std::bind(&BlobDBImpl::ReclaimOpenFiles, this, std::placeholders::_1));
tqueue_.add(static_cast<int64_t>(
bdb_options_.garbage_collection_interval_secs * 1000),
std::bind(&BlobDBImpl::RunGC, this, std::placeholders::_1));
tqueue_.add( tqueue_.add(
kDeleteObsoleteFilesPeriodMillisecs, kDeleteObsoleteFilesPeriodMillisecs,
std::bind(&BlobDBImpl::DeleteObsoleteFiles, this, std::placeholders::_1)); std::bind(&BlobDBImpl::DeleteObsoleteFiles, this, std::placeholders::_1));

Loading…
Cancel
Save