Revert "fix: excluded servers are worst fit for master rather than never assign (so that we can recover if every process has been excluded)"

This reverts commit 83b2ce68c8e1a29fc1559598cc38d3ef7eb46101.
This commit is contained in:
Evan Tschannen 2017-10-01 16:29:32 -07:00
parent 0dde15f1d2
commit 696d432462
1 changed files with 2 additions and 4 deletions

View File

@ -194,10 +194,7 @@ public:
int numEquivalent = 1;
for( auto& it : id_worker ) {
auto fit = it.second.processClass.machineClassFitness( ProcessClass::Master );
if(conf.isExcludedServer(it.second.interf.address())) {
fit = std::max(fit, ProcessClass::WorstFit);
}
if( workerAvailable(it.second, checkStable) && fit != ProcessClass::NeverAssign ) {
if( workerAvailable(it.second, checkStable) && !conf.isExcludedServer(it.second.interf.address()) && fit != ProcessClass::NeverAssign ) {
if( fit < bestFit ) {
bestInfo = std::make_pair(it.second.interf, it.second.processClass);
bestFit = fit;
@ -806,6 +803,7 @@ ACTOR Future<Void> clusterWatchDatabase( ClusterControllerData* cluster, Cluster
iMaster = newMaster.get();
db->masterRegistrationCount = 0;
db->config = DatabaseConfiguration();
db->forceMasterFailure = Promise<Void>();
auto dbInfo = ServerDBInfo( LiteralStringRef("DB") );