serhiy-bzhezytskyy opened a new pull request, #4763: URL: https://github.com/apache/solr/pull/4763
Two members, two remedies, per each one's own deprecation note. `NamedList.NamedListEntry` is fully removed — it added nothing over `AbstractMap.SimpleEntry`, so all 4 production sites move together (3 array creators, 1 filler that stores into them; widening the array's component type is safe by construction, confirmed by a planted mismatch that left tests green and a throwing control that failed them). `MapWriterMap` is NOT removed — its own note says "may keep but use package scope" — so the class and constructor drop `public`; its one caller (`NavigableObject.wrap`) is already in the same package. 4 tests across 4 changed classes, 0 failures. Compile + ecjLint + spotless + renderJavadoc clean on solrj and core. SOLR-18373 (#4761) also touches NamedList.java, in a different region -- simulated the merge order and confirmed this patch still applies cleanly on top of it. 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]
