Confusing that `Set` is called "list". Suggested to call it "hostsInMaintenanceMode" or like that.
Same with `killDataSourceWhitelist` and `killPendingSegmentsSkipList`. Old names could be preserved in the `@JsonProperty` annotations only. Additionally with these two fields, it's very unclear that *data source names* are stored in them. [ Full content available at: https://github.com/apache/incubator-druid/pull/6349 ] This message was relayed via gitbox.apache.org for [email protected]
