janhoy opened a new pull request, #2778: URL: https://github.com/apache/solr/pull/2778
Draft PR to get a green state in branch_9x Tests used to pass on branch_9x a few days ago. Some of the many `SolrCLI` changes touched the urlNormalization stuff so that you needed to add `/solr` at the end of some URLs passed to Solr. In particular this was true for `AssertTool`, which I fix here by getting the SolrClient from SolrCLI util method instead of constructing an `Http2HttpSolrClient` manually. The difference is that the former adds a /solr before constructing the client, and this is consistent with other client initializations in AssertTool. The whole Solr Base Path and `/solr` suffix is a mess, and it is different in main than in branch_9x. So this will keep cropping up for as long as we backport stuff from 10 to 9 :( -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
