Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-8034

If minRF is not satisfied, leader should not put replicas in recovery


    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.4, 6.0
    • Component/s: SolrCloud
    • Labels:
    • Flags:


      If the minimum replication factor parameter (minRf) in a solr update request is not satisfied – i.e. if the update was not successfully applied on at least n replicas where n >= minRf – the shard leader should not put the failed replicas in "leader initiated recovery" and the client should retry the update instead.

      This is so that in the scenario were minRf is not satisfied, the failed replicas can still be eligible to become a leader in case of leader failure, since in the client's perspective this update did not succeed.

      This came up from a network partition scenario where the leader becomes sectioned off from its two followers, but they all could still talk to zookeeper. The partitioned leader set its two followers as in leader initiated recovery, so we couldn't just kill off the partitioned node and have a follower take over leadership. For a minRf=1 case, this is the correct behavior because the partitioned leader would have accepted updates that the followers don't have, and therefore we can't switch leadership or we'd lose those updates. However, in the case of minRf=2, solr never accepted any update in the client's point of view, so in fact the partitioned leader doesn't have any accepted update that the followers don't have, and therefore the followers should be eligible to become leaders. Thus, I'm proposing modifying the leader initiated recovery logic to not put the followers in recovery if the minRf parameter is present and is not satisfied.


        1. SOLR-8034.patch
          9 kB
          Jessica Cheng Mallet
        2. SOLR-8034.patch
          9 kB
          Jessica Cheng Mallet

          Issue Links



              • Assignee:
                anshumg Anshum Gupta
                mewmewball Jessica Cheng Mallet
              • Votes:
                0 Vote for this issue
                5 Start watching this issue


                • Created: