SOLR-18382: remove DocCollection.getReplicas(), migrate 60 call sites - #4760

Open

serhiy-bzhezytskyy wants to merge 17 commits into

apache:mainfrom

serhiy-bzhezytskyy:SOLR-18382-remove-doccollection-getreplicas

Commits on Aug 19, 2026

SOLR-18382: remove DocCollection.getReplicas(), migrate 60 call sites
serhiy-bzhezytskyy
committed
7f8a46f

Commits on Aug 20, 2026

Add DocCollection.getReplicaStream(), dedup 44 call sites
serhiy-bzhezytskyy
committed
acdd050
SOLR-18382: rename DocCollection loop/param vars to collectionState for consistency
serhiy-bzhezytskyy
committed
91fdcf5
SOLR-18382: rewrite a javadoc line that described what it did NOT do
serhiy-bzhezytskyy
committed
803ed76
SOLR-18382: collapse getReplicaForDaemon's double-loop into a Stream one-liner
serhiy-bzhezytskyy
committed
a133f29
SOLR-18382: collapse another predicate to a Stream one-liner
serhiy-bzhezytskyy
committed
02818cb
SOLR-18382: replace manual accumulation with getReplicaStream().toList()
serhiy-bzhezytskyy
committed
0b39c99
SOLR-18382: collect directly into a mutable ArrayList, skip the double allocation
serhiy-bzhezytskyy
committed
5c2fc0c
SOLR-18382: collapse another loop found via census, not flagged by review
serhiy-bzhezytskyy
committed
ffb38a3
SOLR-18382: replace the double-loop with forEach, per David's suggestion
serhiy-bzhezytskyy
committed
a71a0ef
SOLR-18382: replace the double-loop with a Stream, per David's suggestion
serhiy-bzhezytskyy
committed
827b9d7
SOLR-18382: collapse the find-first-then-check predicate to a Stream
serhiy-bzhezytskyy
committed
ed22c64
SOLR-18382: use getReplicaStream().count() for the total-replica count
serhiy-bzhezytskyy
committed
293d6ca
SOLR-18382: use Stream.toList() instead of collect(Collectors.toList())
serhiy-bzhezytskyy
committed
afa2b44
SOLR-18382: rename getReplicaStream() to replicaStream()
serhiy-bzhezytskyy
committed
64a7341
SOLR-18382: pick a random replica by index instead of shuffling
serhiy-bzhezytskyy
committed
aee88ea

Commits on Aug 21, 2026

SOLR-18382: use toList() in getAllNodeNames, inline the single-use state var
serhiy-bzhezytskyy
committed
d1d242c