NextbrickInc opened a new pull request, #4774: URL: https://github.com/apache/solr/pull/4774
## Summary - Curator `RECONNECTED` fires on every ZooKeeper instance hop, so Solr 10 was running full `ZkController` reconnect recovery (re-election, live-node churn, core re-registration) during ZK rolling restarts. - This restores Solr 9 behavior: `OnReconnect` / `OnDisconnect` run only after `ConnectionState.LOST` (session expiry), not after a transient `SUSPENDED` reconnect. - Authored by **Shrey Narayan (NextBrick)**. https://issues.apache.org/jira/browse/SOLR-18298 Related: #4577 already explores this bug with a larger refactor. This PR is the smaller detection-only change requested in review. ## Test plan - [x] `./gradlew :solr:solrj-zookeeper:test --tests org.apache.solr.common.cloud.TestOnReconnectSessionExpiry` - [ ] Confirm a ZK rolling restart no longer logs “refreshing core states after session expiration” unless the session actually expired Made with [Cursor](https://cursor.com) -- 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]
