lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrzej Bialecki (JIRA)" <>
Subject [jira] [Commented] (SOLR-2822) don't run update processors twice
Date Thu, 17 May 2012 22:11:16 GMT


Andrzej Bialecki  commented on SOLR-2822:

This looks good. Some comments:
* javadocs should mention somewhere (UpdateProcessorRequestChain) the skipping of parts of
the chain when FROMLEADER is seen.
* tests don't seem to cover any case that uses requests with TOLEADER set.
* re. comment in UpdateRequestProcessorChain, where DUPFactory() is inserted if missing: I'd
say if someone forgot or is unaware of the need for a distrib processor then it should be
inserted as it is done in the patch - right before the run processor, so that all modifications
to documents are completed before distributing (modifications could affect the shard #).
* minor nits:
** BasicDistributedZkTest.testUpdateProcessorsRunOnlyOnce() has a hanging :TODO:, not sure
what it refers to?
** DistributingUpdateProcessorFactory: typos "not inteded", "skiped".
** DistributedUpdateProcessor: typo "exposted"
** UpdateRequestProcessorChain: mentions DistributedUpdateProcessorFactoryMarker, "then and".
> don't run update processors twice
> ---------------------------------
>                 Key: SOLR-2822
>                 URL:
>             Project: Solr
>          Issue Type: Sub-task
>          Components: SolrCloud, update
>            Reporter: Yonik Seeley
>             Fix For: 4.0
>         Attachments: SOLR-2822.patch
> An update will first go through processors until it gets to the point where it is forwarded
to the leader (or forwarded to replicas if already on the leader).
> We need a way to skip over the processors that were already run (perhaps by using a processor
chain dedicated to sub-updates?

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message