[jira] [Commented] (SOLR-4613) Move checkDistributed to SearchHandler

["Mark Miller (JIRA)" <ji...@apache.org>]

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

Mark Miller commented on SOLR-4613:
-----------------------------------

I think part of the motivation for this was to allow a single pluggable point for controlling shard selection.

Been a while since I've looked closely, but this API has always looked like it could use some refactoring.

Hope to be able to take a closer look later.
                
> Move checkDistributed to SearchHandler
> --------------------------------------
>
>                 Key: SOLR-4613
>                 URL: https://issues.apache.org/jira/browse/SOLR-4613
>             Project: Solr
>          Issue Type: Improvement
>          Components: search
>            Reporter: Ryan Ernst
>         Attachments: SOLR-4613.patch
>
>
> Currently a ShardHandler is created for a request even for non distributed requests.  The checkDistributed function on ShardHandler has no special state kept in the ShardHandler.  Historically it used to be in QueryComponent, but it seems like SearchHandler would be the right place.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org