dsmiley commented on PR #2459: URL: https://github.com/apache/solr/pull/2459#issuecomment-2110379724
Separately, I wonder if we make it too easy to loop the state of every collection. I'm looking at `Cluster` added by @murblanc which contains not only an `Iterator<SolrCollection> iterator();` method, but also `Iterable<SolrCollection> collections();` to make it that much easier. Instead, let's just have a method to list collection names. Then if the caller is hell bent on looping everything in the cluster, it's going to be that much more obvious to that code that it's looking up collection info for each and every one. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
