[ 
https://issues.apache.org/jira/browse/SOLR-8298?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christine Poerschke updated SOLR-8298:
--------------------------------------
    Attachment: SOLR-8298.patch

Attaching proposed patch against trunk.

[~sachingoyal] and [~thelabdude] if you would have a chance to take a look 
perhaps at some point that would be much appreciated. There should be no actual 
change here compared to the SOLR-6832 logic i.e. this is only a small 
implementation refactor. Thank you.

> small preferLocalShards implementation refactor
> -----------------------------------------------
>
>                 Key: SOLR-8298
>                 URL: https://issues.apache.org/jira/browse/SOLR-8298
>             Project: Solr
>          Issue Type: Wish
>            Reporter: Christine Poerschke
>            Assignee: Christine Poerschke
>            Priority: Minor
>         Attachments: SOLR-8298.patch
>
>
> Towards rebasing the SOLR-6730 patch after SOLR-6832 and other changes - 
> proposed patch against trunk to follow.
> existing calling chain:
>  * {{ResponseBuilder.addRequest(... ShardRequest sreq)}} does {{sreq.rb = 
> this;}} so that later on {{HttpShardHandler.submit(ShardRequest sreq ...)}} 
> can do {{sreq.rb.req.getOriginalParams().getBool}} for 
> {{CommonParams.PREFER_LOCAL_SHARDS}}
> proposed alternative calling chain:
>  * {{HttpShardHandler.prepDistributed(ResponseBuilder rb)}} sets 
> {{rb.preferredHostAddress}} and {{SearchHandler}} calls 
> {{ShardHandler.submit(ShardRequest sreq ... rb.preferredHostAddress)}}
> structural changes:
>  * {{ShardRequest.rb}} member removed in favour of a new 
> {{ResponseBuilder.preferredHostAddress}} member.
>  * {{String preferredHostAddress}} argument added to the abstract 
> {{ShardHandler.submit}} method (and to two derived (test) classes' submit 
> methods also).
>  * {code}public void submit(ShardRequest sreq, String shard, 
> ModifiableSolrParams params) { submit(sreq, shard, params, null); } {code} 
> added to avoid having to change {{ShardHandler.submit}} callers which don't 
> have a concept of preferring a local shard e.g. for PeerSync requests.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to