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

fix: isExcluded and isCleared were exactly wrong 

fix: isCleared should mean the process is dead
This commit is contained in:
Evan Tschannen 2018-07-05 02:22:22 -07:00
parent 507b3bacb0
commit 7315e5da55
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
fdbrpc/sim2.actor.cpp
View File

@ -1266,7 +1266,7 @@ public:
processesDead.push_back(processInfo); processesDead.push_back(processInfo);
excluded++; excluded++;
} }
else if (!processInfo->isCleared()) { else if (processInfo->isCleared()) {
processesDead.push_back(processInfo); processesDead.push_back(processInfo);
cleared++; cleared++;
} }

4
fdbrpc/simulator.h
View File

@ -77,8 +77,8 @@ public:
bool isReliable() const { return !failed && fault_injection_p1 == 0 && fault_injection_p2 == 0; } bool isReliable() const { return !failed && fault_injection_p1 == 0 && fault_injection_p2 == 0; }
bool isAvailable() const { return !isExcluded() && isReliable(); } bool isAvailable() const { return !isExcluded() && isReliable(); }
bool isExcluded() const { return !excluded; } bool isExcluded() const { return excluded; }
bool isCleared() const { return !cleared; } bool isCleared() const { return cleared; }
// Returns true if the class represents an acceptable worker // Returns true if the class represents an acceptable worker
bool isAvailableClass() const { bool isAvailableClass() const {