[ https://issues.apache.org/jira/browse/SOLR-6691?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Erick Erickson updated SOLR-6691: --------------------------------- Description: The original code (SOLR-6517) assumed that changes in the clusterstate after issuing a command to the overseer to change the leader indicated that the leader was successfully changed. Fortunately, Noble clued me in that this isn't the case and that the potential leader needs to insert itself in the leader election queue before trigging the change leader command. Inserting themselves in the front of the queue should probably happen in BALANCESHARDUNIQUE when the preferredLeader property is assigned as well. [~noble.paul] Do evil things happen if a node joins at the head but it's _already_ in the queue? These ephemeral nodes in the queue are watching each other. So if node1 is the leader you have node1 <- node2 <- node3 <- node4 where <- means "watches". Now, if node3 puts itself at the head of the list, you have {code} node1 <- node2 <- node3 <- node4 {code} I _think_ when I was looking at this it all "just worked". 1> node 1 goes down. Nodes 2 and 3 duke it out but there's code to insure that node3 becomes the leader and node2 inserts itself at then end so it's watching node 4. 2> node 2 goes down, nobody gets notified and it doesn't matter. 3> node 3 goes down, node 4 gets notified and starts watching node 2 by inserting itself at the end of the list. 4> node 4 goes down, nobody gets notified and it doesn't matter. was: The original code (SOLR-6517) assumed that changes in the clusterstate after issuing a command to the overseer to change the leader indicated that the leader was successfully changed. Fortunately, Noble clued me in that this isn't the case and that the potential leader needs to insert itself in the leader election queue before trigging the change leader command. Inserting themselves in the front of the queue should probably happen in BALANCESHARDUNIQUE when the preferredLeader property is assigned as well. [~noble.paul] Do evil things happen if a node joins at the head but it's _already_ in the queue? These ephemeral nodes in the queue are watching each other. So if node1 is the leader you have node1 <- node2 <- node3 <- node4 where <- means "watches". Now, if node3 puts itself at the head of the list, you have {code} node1 <- node2 <- node3 <- node4 {code} I _think_ when I was looking at this it all "just worked". 1> node 1 goes down. Nodes 2 and 3 duke it out but there's code to insure that node3 becomes the leader and node2 inserts itself at then end so it's watching node 4. 2> node 2 goes down, nobody gets notified and it doesn't matter. 3> node 3 goes down, node 4 gets notified and starts watching node 2 by inserting itself at the end of the list. 4> node 4 goes down, nobody gets notified and it doesn't matter. > REBALANCELEADERS needs to change the leader election queue. > ----------------------------------------------------------- > > Key: SOLR-6691 > URL: https://issues.apache.org/jira/browse/SOLR-6691 > Project: Solr > Issue Type: Bug > Reporter: Erick Erickson > Assignee: Erick Erickson > > The original code (SOLR-6517) assumed that changes in the clusterstate after > issuing a command to the overseer to change the leader indicated that the > leader was successfully changed. Fortunately, Noble clued me in that this > isn't the case and that the potential leader needs to insert itself in the > leader election queue before trigging the change leader command. > Inserting themselves in the front of the queue should probably happen in > BALANCESHARDUNIQUE when the preferredLeader property is assigned as well. > [~noble.paul] Do evil things happen if a node joins at the head but it's > _already_ in the queue? These ephemeral nodes in the queue are watching each > other. So if node1 is the leader you have > node1 <- node2 <- node3 <- node4 > where <- means "watches". > Now, if node3 puts itself at the head of the list, you have > {code} > node1 <- node2 > <- node3 <- node4 > {code} > I _think_ when I was looking at this it all "just worked". > 1> node 1 goes down. Nodes 2 and 3 duke it out but there's code to insure > that node3 becomes the leader and node2 inserts itself at then end so it's > watching node 4. > 2> node 2 goes down, nobody gets notified and it doesn't matter. > 3> node 3 goes down, node 4 gets notified and starts watching node 2 by > inserting itself at the end of the list. > 4> node 4 goes down, nobody gets notified and it doesn't matter. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org