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

David Smiley commented on SOLR-16505:
-------------------------------------

The word "executor" in this file is an overloaded word.  Generally we refer to 
an ExecutorService.  But the HttpClients don't use an ExecutorService (ignoring 
that the Solr Http2SolrClient using Jetty can use one for async requests; isn't 
relevant in UpdateShardHandler).  The Apache one does use an 
httpRequestExecutor but this is *not* an ExecutorService; it's a mechanism to 
decorate the HTTP requests, like for instrumentation.  It's not used for 
multi-threaded execution.  There's a Jetty side equivalent you will see in this 
file: InstrumentedHttpListenerFactory.

> Switch UpdateShardHandler.getRecoveryOnlyHttpClient to Jetty HTTP2
> ------------------------------------------------------------------
>
>                 Key: SOLR-16505
>                 URL: https://issues.apache.org/jira/browse/SOLR-16505
>             Project: Solr
>          Issue Type: Sub-task
>            Reporter: David Smiley
>            Priority: Major
>
> This method and its callers (only RecoveryStrategy) should be converted to a 
> Jetty HTTP2 client.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to