I don't feel like the `All` suffix is necessary for these methods, it seems implied since the url path doesn't specify a supervisor id, and doesn't appear to be a common convention in other API paths that do things to multiple things. That said, the base path for the supervisor api probably should have been `/druid/indexer/v1/supervisors` instead of `/druid/indexer/v1/supervisor` so it would have read a bit better, but that seems more painful than useful to change
[ Full content available at: https://github.com/apache/incubator-druid/pull/6272 ] This message was relayed via gitbox.apache.org for [email protected]
