HoustonPutman opened a new pull request, #3398:
URL: https://github.com/apache/solr/pull/3398

   https://issues.apache.org/jira/browse/SOLR-17792
   
   This is a WIP but we really should be able to get rid of the synchronization 
and make this logic so much simpler.
   
   Here I'm making `responseFutureMap` a ConcurrentMap, so it doesn't need to 
be synchronized. Synchronization blocks also won't be necessary because we 
won't be using `pending` anymore. We will consider ourselves pending if 
`responseFutureMap` or `responses` is not empty. (previously pending was 
increased if either something was added to `responseFutureMap`, or a 
no-shard-response was added to `responses`).
   
   This is a blocker for 9.9, because the ParallelHttpShardHandler is unusable 
without fixing this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to