[ 
https://issues.apache.org/jira/browse/SOLR-8368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15049172#comment-15049172
 ] 

Mike Drob commented on SOLR-8368:
---------------------------------

Remove the TODO about doing replay before leader election?

Could put ulog declaration in replayTransactionLogIfNeeded, cut down on a 
method parameter and some scope that way.

bq. The only change beyond calling tlog replay before entering the election is 
that we have to remove an || isLeader check against it since it happens before 
we know if we will be the leader or not.
This means that a non-leader in shard construction state will no longer do a 
replay. I don't know enough about how shard splits work to know if this is ok, 
but I think the impact is that replicas will take longer to come online since 
they will need to do an additional recovery once the sub-shard leader is 
active. If that's the case then the rest of the patch LGTM.

I see a couple of minor places where we could clarify things, but it's not 
worth it for this issue.

> A SolrCore needs to replay it's tlog before the leader election process.
> ------------------------------------------------------------------------
>
>                 Key: SOLR-8368
>                 URL: https://issues.apache.org/jira/browse/SOLR-8368
>             Project: Solr
>          Issue Type: Bug
>            Reporter: Mark Miller
>            Assignee: Mark Miller
>         Attachments: SOLR-8368.patch
>
>
> If we do it after like now, the correct leader may not be able to become 
> leader.



--
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

Reply via email to