It breaks the contract of `DruidCluster`, in a way. Benignantly at the moment, but it could break. Constructors shouldn't just accept a collection of collections as a parameter and assign to a field without defensive copy / invariant check. Performance is not an issue, since this constructor is used only in tests.
[ Full content available at: https://github.com/apache/incubator-druid/pull/6349 ] This message was relayed via gitbox.apache.org for [email protected]
