Simplifying code
This commit is contained in:
@@ -152,7 +152,6 @@ TaskFuture DaggyRunnerTaskExecutor::execute(DAGRunID runID,
|
|||||||
auto taskUsed = capacityFromTask(task);
|
auto taskUsed = capacityFromTask(task);
|
||||||
|
|
||||||
std::string exe_runner;
|
std::string exe_runner;
|
||||||
Capacity *exe_capacity;
|
|
||||||
|
|
||||||
// Block until a host is found
|
// Block until a host is found
|
||||||
std::unique_lock<std::mutex> lock(runnersGuard_);
|
std::unique_lock<std::mutex> lock(runnersGuard_);
|
||||||
@@ -161,16 +160,16 @@ TaskFuture DaggyRunnerTaskExecutor::execute(DAGRunID runID,
|
|||||||
for (auto &[runner, capacity] : runners_) {
|
for (auto &[runner, capacity] : runners_) {
|
||||||
if (capacity.cores >= taskUsed.cores and
|
if (capacity.cores >= taskUsed.cores and
|
||||||
capacity.memoryMB >= taskUsed.memoryMB) {
|
capacity.memoryMB >= taskUsed.memoryMB) {
|
||||||
exe_runner = runner;
|
exe_runner = runner;
|
||||||
exe_capacity = &capacity;
|
capacity.cores -= taskUsed.cores;
|
||||||
|
capacity.memoryMB -= taskUsed.memoryMB;
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return false;
|
return false;
|
||||||
});
|
});
|
||||||
exe_capacity->cores -= taskUsed.cores;
|
|
||||||
exe_capacity->memoryMB -= taskUsed.memoryMB;
|
|
||||||
|
|
||||||
|
// Capacity has already been reduced
|
||||||
lock.unlock();
|
lock.unlock();
|
||||||
|
|
||||||
std::stringstream ss;
|
std::stringstream ss;
|
||||||
|
|||||||
Reference in New Issue
Block a user