serhiy-bzhezytskyy opened a new pull request, #4766: URL: https://github.com/apache/solr/pull/4766
Removes both `Utils.getBaseUrlForNodeName` overloads — pure delegates to `URLUtil.getBaseUrlForNodeName`, 24 call sites (4 production, 20 test) moved with identical semantics. `Utils.java` now has zero deprecated members. Where to look: the name collides three ways — the removed `Utils` static, its `URLUtil` replacement, and `ZkStateReader`'s unrelated one-arg *instance* method (~60 call sites, untouched, already delegates to `URLUtil` directly). Census had to key off the exact static signature, not the name. The rename is 5 chars longer, pushing 11 lines from 100 to 102 chars — reflowed by `google-java-format`, matched against pre-existing breaks in the same files. Guard: `URLUtilTest` pins 4 literal URLs; a planted fault caught before and after the migration, so coverage survived. 9 changed test classes, 52 tests, 0 failures. All gates green. SOLR-18373 (#4761) and SOLR-18380 (#4762) both touch CloudSolrClient.java -- simulated both merged together, applies cleanly. SOLR-18388 (local, not yet a PR) shares ZkStateReaderTest.java. 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]
