murblanc commented on code in PR #2834:
URL: https://github.com/apache/solr/pull/2834#discussion_r1827308183
##########
solr/solrj/src/java/org/apache/solr/common/cloud/ClusterState.java:
##########
@@ -382,6 +383,7 @@ void setLiveNodes(Set<String> liveNodes) {
* Be aware that this may return collections which may not exist now. You
can confirm that this
* collection exists after verifying CollectionRef.get() != null
*/
+ @Deprecated // see collectionStream()
Review Comment:
And IMO this can't be deprecated for performance reasons until
`getCollectionNames()` is added to replace it.
--
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
