dsmiley commented on PR #2811:
URL: https://github.com/apache/solr/pull/2811#issuecomment-2450153694
I know it's out of scope... sort of... but @iamsanjay recently updated
SolrClientNodeStateProvider.invoke to not call setBasePath but the result is
more lines of code / complexity than needed. There only needs to be one line
of code inside the try-catch as follows:
return cloudSolrClient.getHttpClient().requestWithBaseUrl(url,
request::process);
If you think this could reasonably be in this PR then please update (or tell
me to add it accordingly). Or...
--
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]
