2019-12-16 22:28:06 +00:00
|
|
|
// Copyright (c) 2011-present, Facebook, Inc. All rights reserved.
|
|
|
|
// This source code is licensed under both the GPLv2 (found in the
|
|
|
|
// COPYING file in the root directory) and Apache 2.0 License
|
|
|
|
// (found in the LICENSE.Apache file in the root directory).
|
|
|
|
//
|
|
|
|
// Copyright (c) 2011 The LevelDB Authors. All rights reserved.
|
|
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
|
|
// found in the LICENSE file. See the AUTHORS file for names of contributors.
|
|
|
|
|
|
|
|
#ifdef GFLAGS
|
|
|
|
#pragma once
|
|
|
|
#include "db_stress_tool/db_stress_common.h"
|
|
|
|
|
2020-02-20 20:07:53 +00:00
|
|
|
namespace ROCKSDB_NAMESPACE {
|
2019-12-16 22:28:06 +00:00
|
|
|
class DbStressEnvWrapper : public EnvWrapper {
|
|
|
|
public:
|
|
|
|
explicit DbStressEnvWrapper(Env* t) : EnvWrapper(t) {}
|
2022-01-05 00:44:54 +00:00
|
|
|
static const char* kClassName() { return "DbStressEnv"; }
|
|
|
|
const char* Name() const override { return kClassName(); }
|
2019-12-16 22:28:06 +00:00
|
|
|
|
|
|
|
Status DeleteFile(const std::string& f) override {
|
|
|
|
// We determine whether it is a manifest file by searching a strong,
|
|
|
|
// so that there will be false positive if the directory path contains the
|
|
|
|
// keyword but it is unlikely.
|
Fix, enable, and enhance backup/restore in db_stress (#7348)
Summary:
Although added to db_stress, testing of backup/restore
was never integrated into the crash test, originally concerned about
performance. I've enabled it now and to address the peformance concern,
testing backup/restore is always skipped once the db exceeds a certain
size threshold, default 100MB. This should provide sufficient
opportunity for testing BackupEngine without bogging down everything
else with heavier and heavier operations.
Also fixed backup/restore in db_stress by making sure PurgeOldBackups
can remove manifest files, which are normally kept around for db_stress.
Added more coverage of backup options, and up to three backups being
saved in one backup directory (in some cases).
Pull Request resolved: https://github.com/facebook/rocksdb/pull/7348
Test Plan:
ran 'make blackbox_crash_test' for a while, with heightened
probabilitly of taking backups (1/10k). Also confirmed with some debug
output that the code is being covered, TestBackupRestore only takes
a few seconds to complete when triggered, and even at 1/10k and ~50MB
database, there's <,~ 1 thread testing backups at any time.
Reviewed By: ajkr
Differential Revision: D23510835
Pulled By: pdillinger
fbshipit-source-id: b6b8735591808141f81f10773ac31634cf03b6c0
2020-09-04 03:11:45 +00:00
|
|
|
// Checkpoint, backup, and restore directories needs to be exempted.
|
2019-12-16 22:28:06 +00:00
|
|
|
if (!if_preserve_all_manifests ||
|
2019-12-21 06:08:44 +00:00
|
|
|
f.find("MANIFEST-") == std::string::npos ||
|
Fix, enable, and enhance backup/restore in db_stress (#7348)
Summary:
Although added to db_stress, testing of backup/restore
was never integrated into the crash test, originally concerned about
performance. I've enabled it now and to address the peformance concern,
testing backup/restore is always skipped once the db exceeds a certain
size threshold, default 100MB. This should provide sufficient
opportunity for testing BackupEngine without bogging down everything
else with heavier and heavier operations.
Also fixed backup/restore in db_stress by making sure PurgeOldBackups
can remove manifest files, which are normally kept around for db_stress.
Added more coverage of backup options, and up to three backups being
saved in one backup directory (in some cases).
Pull Request resolved: https://github.com/facebook/rocksdb/pull/7348
Test Plan:
ran 'make blackbox_crash_test' for a while, with heightened
probabilitly of taking backups (1/10k). Also confirmed with some debug
output that the code is being covered, TestBackupRestore only takes
a few seconds to complete when triggered, and even at 1/10k and ~50MB
database, there's <,~ 1 thread testing backups at any time.
Reviewed By: ajkr
Differential Revision: D23510835
Pulled By: pdillinger
fbshipit-source-id: b6b8735591808141f81f10773ac31634cf03b6c0
2020-09-04 03:11:45 +00:00
|
|
|
f.find("checkpoint") != std::string::npos ||
|
|
|
|
f.find(".backup") != std::string::npos ||
|
|
|
|
f.find(".restore") != std::string::npos) {
|
2019-12-16 22:28:06 +00:00
|
|
|
return target()->DeleteFile(f);
|
|
|
|
}
|
2021-04-28 17:57:11 +00:00
|
|
|
// Rename the file instead of deletion to keep the history, and
|
|
|
|
// at the same time it is not visible to RocksDB.
|
|
|
|
return target()->RenameFile(f, f + "_renamed_");
|
2019-12-16 22:28:06 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// If true, all manifest files will not be delted in DeleteFile().
|
|
|
|
bool if_preserve_all_manifests = true;
|
|
|
|
};
|
2020-02-20 20:07:53 +00:00
|
|
|
} // namespace ROCKSDB_NAMESPACE
|
2019-12-16 22:28:06 +00:00
|
|
|
#endif // GFLAGS
|