lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gregory Chanan (JIRA)" <>
Subject [jira] [Commented] (SOLR-9199) ZkController#publishAndWaitForDownStates logic is inefficient
Date Thu, 09 Jun 2016 00:40:20 GMT


Gregory Chanan commented on SOLR-9199:

lgtm.  Could maybe quibble that the old code wouldn't print "timing out" if it never entered
the loop (e.g. if wait time was 0), but that doesn't seem right or wrong.

I'll commit to trunk and 6.x assuming the precommit and tests pass.

> ZkController#publishAndWaitForDownStates logic is inefficient
> -------------------------------------------------------------
>                 Key: SOLR-9199
>                 URL:
>             Project: Solr
>          Issue Type: Bug
>            Reporter: Hrishikesh Gadre
>            Assignee: Hrishikesh Gadre
>         Attachments: SOLR-9199.patch
> The following logic introduced as part of SOLR-8720 is inefficient. 
> Specifically,
> * foundStates flag is set to TRUE before the for loop.
> * In the for loop we check if any replica on this node is not in the DOWN state. If yes,
then foundStates = FALSE
> * If foundStates == TRUE then we break out of the loop and return.
> The problem here is that once foundStates is set to FALSE, it is never reset to TRUE.
Hence we end up spending the whole 60 secs iterating over the collections even though all
the replicas are marked as DOWN in later iterations.

This message was sent by Atlassian JIRA

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

View raw message