Additionally, for a hypothetical `suspendOrResumeAllSupervisors` method,  it 
might also be nice to extract the actual logic from `suspendOrResumeSupervisor` 
into another method, `possiblySuspendOrResumeSupervisor` that doesn't have the 
precondition checks, similar to the function of 
`possiblyStopAndRemoveSupervisorInternal`, and call that method from both 
`suspendOrResumeSupervisor` and `suspendOrResumeSupervisors`.

[ Full content available at: 
https://github.com/apache/incubator-druid/pull/6272 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to