[
https://issues.apache.org/jira/browse/SOLR-8288?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15004799#comment-15004799
]
Mike Drob commented on SOLR-8288:
---------------------------------
bq. + &&
!stdNode.getNodeProps().getCoreUrl().equals(leaderProps.getCoreUrl())) { // we
do not want to put ourself into LIR
If we are comparing URLs, then would it make sense to check against the
[replicaURL|https://github.com/apache/lucene-solr/blob/eab11f7fe242710216786db70814a0f492342b38/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java#L824]
that we saw the error on?
{{replicaUrl.equals(leaderProps.getCoreUrl())}} instead?
> DistributedUpdateProcessor#doFinish should explicitly check and ensure it
> does not try to put itself into LIR.
> --------------------------------------------------------------------------------------------------------------
>
> Key: SOLR-8288
> URL: https://issues.apache.org/jira/browse/SOLR-8288
> Project: Solr
> Issue Type: Bug
> Reporter: Mark Miller
> Assignee: Mark Miller
> Fix For: 5.4, Trunk
>
> Attachments: SOLR-8288.patch
>
>
> We have to be careful about this because currently, something like a commit
> is sent over http even to the local node and if that fails for some reason,
> the leader might try and LIR itself.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
