The reason why I removed this is that I have moved the condition `spec.get().isSuspended() == suspend` into `SupervisorManager#possiblySuspendOrResumeSupervisorInternal` and modified the implementation of `SupervisorResource#specSuspendOrResume` to keep consistent with the implementation of `SupervisorResource#terminate`. After done these, I have to modify the test code, then I found there was no need to keep this. You can check the whole `SupervisorResourceTest#testSpecSuspend` then you'll find I have tested both behaviors.
[ Full content available at: https://github.com/apache/incubator-druid/pull/6272 ] This message was relayed via gitbox.apache.org for [email protected]
