Previously `servers` was a lazy iterable, that converted `DruidServer` objects to `ImmutableDruidServer`s on each iteration, 2 or 3 times, if debug logging is enabled. I think it could lead to massive garbage production on coordinator, because each `toImmutableDruidServer()` call moves a lot of inventory to immutable collections.
[ Full content available at: https://github.com/apache/incubator-druid/pull/6385 ] This message was relayed via gitbox.apache.org for [email protected]
