aparnasuresh85 commented on code in PR #2571:
URL: https://github.com/apache/solr/pull/2571#discussion_r1690487661
##########
solr/solrj/src/test/org/apache/solr/client/solrj/impl/CloudHttp2SolrClientTest.java:
##########
@@ -246,6 +257,81 @@ public void testAliasHandling() throws Exception {
2, client.query(null,
paramsWithMixedCollectionAndAlias).getResults().getNumFound());
}
+ @Test
+ @LogLevel("org.apache.solr.servlet.HttpSolrCall=DEBUG")
+ public void testHttpCspPerf() throws Exception {
+
+ performTest(false, "HTTPCSPTEST", 1, 10, 15, 15);
+
+ }
+ @Test
+ @LogLevel("org.apache.solr.servlet.HttpSolrCall=DEBUG")
Review Comment:
Not a great design for the tests I admit - I wanted to reuse code.
I also added a test using Zk based CSP - to compare metrics against the http
one
After making the updates - I notice http based CSC also making a call to
Solr to LISTALIASES (made obvious by the updated test)
The log listening can be consolidated somewhat.
--
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
