pvcnt commented on code in PR #2313:
URL: https://github.com/apache/solr/pull/2313#discussion_r1573262174
##########
solr/solrj/src/java/org/apache/solr/client/solrj/impl/Http2SolrClient.java:
##########
@@ -1048,7 +1056,7 @@ private static class AsyncTracker {
int getMaxRequestsQueuedPerDestination() {
Review Comment:
This is used to set Jetty's `maxRequestsQueuedPerDestination`, which is
indeed per host. But the AsyncTracker itself has no per-host logic. However, it
depends in which context it is used. When used by the LbSolrClient (notably in
the context of the HttpShardHandler, but only when used directly by any
client), there is effectively one Http2SolrClient created per host, so this
matches. In other contexts, I believe there is a mismatch and it doesn't do
what it claims to do.
--
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: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
