anshumg commented on pull request #2010: URL: https://github.com/apache/lucene-solr/pull/2010#issuecomment-725022361
We should restrict this PR to just handle the removal of persisted base_url in ZK. For users who intend to enable/disable TLS for their cluster, they should plan for downtime and as guaranteeing a safe rolling restart during such a transition is not possible right now. I think we're better off not trying for a best effort during a rolling restart here and removing the live_nodes part of the code change to keep things simple. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
