wbtiger
/
foundationdb
mirror of https://github.com/apple/foundationdb.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FastRestore:Disable addPrefix test

This commit is contained in:
Meng Xu 2020-06-24 14:26:33 -07:00
parent b69c36f0d6
commit a0edd1d4eb
2 changed files with 26 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
fdbserver/workloads/AtomicRestore.actor.cpp
View File

@ -48,19 +48,21 @@ struct AtomicRestoreWorkload : TestWorkload {
addPrefix = getOption(options, LiteralStringRef("addPrefix"), LiteralStringRef(""));
removePrefix = getOption(options, LiteralStringRef("removePrefix"), LiteralStringRef(""));
// Correctness is not clean for addPrefix feature yet. Uncomment below to enable the test
// Generate addPrefix
if (addPrefix.size() == 0 && removePrefix.size() == 0) {
if (deterministicRandom()->random01() < 0.5) { // Generate random addPrefix
int len = deterministicRandom()->randomInt(1, 100);
std::string randomStr = deterministicRandom()->randomAlphaNumeric(len);
TraceEvent("AtomicRestoreWorkload")
.detail("GenerateAddPrefix", randomStr)
.detail("Length", len)
.detail("StrLen", randomStr.size());
addPrefix = Key(randomStr);
}
}
// if (addPrefix.size() == 0 && removePrefix.size() == 0) {
// if (deterministicRandom()->random01() < 0.5) { // Generate random addPrefix
// int len = deterministicRandom()->randomInt(1, 100);
// std::string randomStr = deterministicRandom()->randomAlphaNumeric(len);
// TraceEvent("AtomicRestoreWorkload")
// .detail("GenerateAddPrefix", randomStr)
// .detail("Length", len)
// .detail("StrLen", randomStr.size());
// addPrefix = Key(randomStr);
// }
// }
TraceEvent("AtomicRestoreWorkload").detail("AddPrefix", addPrefix).detail("RemovePrefix", removePrefix);
ASSERT(addPrefix.size() == 0 && removePrefix.size() == 0);
// Do not support removePrefix right now because we must ensure all backup keys have the removePrefix
// otherwise, test will fail because fast restore will simply add the removePrefix to every key in the end.
ASSERT(removePrefix.size() == 0);

24
fdbserver/workloads/BackupAndParallelRestoreCorrectness.actor.cpp
View File

@ -82,21 +82,23 @@ struct BackupAndParallelRestoreCorrectnessWorkload : TestWorkload {
KeyRef endRange;
UID randomID = nondeterministicRandom()->randomUniqueID();
// Correctness is not clean for addPrefix feature yet. Uncomment below to enable the test
// Generate addPrefix
if (addPrefix.size() == 0 && removePrefix.size() == 0) {
if (deterministicRandom()->random01() < 0.5) { // Generate random addPrefix
int len = deterministicRandom()->randomInt(1, 100);
std::string randomStr = deterministicRandom()->randomAlphaNumeric(len);
TraceEvent("BackupAndParallelRestoreCorrectness")
.detail("GenerateAddPrefix", randomStr)
.detail("Length", len)
.detail("StrLen", randomStr.size());
addPrefix = Key(randomStr);
}
}
// if (addPrefix.size() == 0 && removePrefix.size() == 0) {
// if (deterministicRandom()->random01() < 0.5) { // Generate random addPrefix
// int len = deterministicRandom()->randomInt(1, 100);
// std::string randomStr = deterministicRandom()->randomAlphaNumeric(len);
// TraceEvent("BackupAndParallelRestoreCorrectness")
// .detail("GenerateAddPrefix", randomStr)
// .detail("Length", len)
// .detail("StrLen", randomStr.size());
// addPrefix = Key(randomStr);
// }
// }
TraceEvent("BackupAndParallelRestoreCorrectness")
.detail("AddPrefix", addPrefix)
.detail("RemovePrefix", removePrefix);
ASSERT(addPrefix.size() == 0 && removePrefix.size() == 0);
// Do not support removePrefix right now because we must ensure all backup keys have the removePrefix
// otherwise, test will fail because fast restore will simply add the removePrefix to every key in the end.
ASSERT(removePrefix.size() == 0);