Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix incorrect precondition check in `SupervisorManager.suspendOrResumeSupervisor` (#6364) 

This check is reverse from the intention
This commit is contained in:
Clint Wylie 2018-09-21 17:40:14 -07:00 committed by Jonathan Wei
parent f12ffd19a8
commit 399a5659b2
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
indexing-service/src/main/java/org/apache/druid/indexing/overlord/supervisor/SupervisorManager.java
View File

@ -95,7 +95,7 @@ public class SupervisorManager
{ {
Preconditions.checkState(started, "SupervisorManager not started"); Preconditions.checkState(started, "SupervisorManager not started");
Pair<Supervisor, SupervisorSpec> pair = supervisors.get(id); Pair<Supervisor, SupervisorSpec> pair = supervisors.get(id);
Preconditions.checkNotNull(pair.lhs, "spec"); Preconditions.checkNotNull(pair.rhs, "spec");
synchronized (lock) { synchronized (lock) {
Preconditions.checkState(started, "SupervisorManager not started"); Preconditions.checkState(started, "SupervisorManager not started");
SupervisorSpec nextState = suspend ? pair.rhs.createSuspendedSpec() : pair.rhs.createRunningSpec(); SupervisorSpec nextState = suspend ? pair.rhs.createSuspendedSpec() : pair.rhs.createRunningSpec();