risdenk commented on code in PR #1445:
URL: https://github.com/apache/solr/pull/1445#discussion_r1134406595


##########
solr/benchmark/src/java/org/apache/solr/bench/MiniClusterState.java:
##########
@@ -273,7 +273,7 @@ public void startMiniCluster(int nodeCount) {
         nodes.add(runner.getBaseUrl().toString());
       }
 
-      client = new Http2SolrClient.Builder().build();
+      client = new Http2SolrClient.Builder().useHttp1_1(true).build();

Review Comment:
   so I pulled down your branch and am seeing the same exception you shared.
   
   ```
   ./jmh.sh -jvmArgsAppend 
'-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=*:5005' 
search.SimpleSearch
   ```
   
   It looks like to me the exception is coming from `HttpTransportOverHTTP2` 
line 375 - 
https://github.com/eclipse/jetty.project/blob/jetty-10.0.x/jetty-http2/http2-server/src/main/java/org/eclipse/jetty/http2/server/HttpTransportOverHTTP2.java#L375
   
   I don't know why or how we get into this scenario but it seems like some 
sort of race condition or something since if you slow down the benchmark (ie: 
add a bunch of debug logging for Jetty) then things seem to be ok.



-- 
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

Reply via email to