serhiy-bzhezytskyy commented on PR #4762: URL: https://github.com/apache/solr/pull/4762#issuecomment-5346814027
@dsmiley you deprecated all of this two months ago (`CloudSolrClient.connect()` x2, the legacy `Builder`, `ClusterStateProvider.connect()`), so it's fresh enough to actually need your take. 54 sites across 5 modules, and migrating them turned up two places where the removal could've silently broken a test (a mock returning null -> 150s hang; a broken `var` swallowing a compile error) -- fixed, but a second pair of eyes from whoever wrote the deprecation would be reassuring. AI-assisted (Claude Sonnet 5) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
