The caller needs only `DruidServer`, not entire map or `QueryableDruidServer`. 
Also, it's much better to not return mutable objects. Please change this to 
like below:

```java
return clients.values().stream()
           .map(queryableDruidServer -> 
queryableDruidServer.getServer().toImmutableDruidServer())
           .collect(Collectors.toList());
```

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

Reply via email to