epugh commented on code in PR #2378:
URL: https://github.com/apache/solr/pull/2378#discussion_r1570775062
##########
solr/solrj/src/java/org/apache/solr/client/solrj/impl/CloudHttp2SolrClient.java:
##########
@@ -422,7 +422,10 @@ public CloudHttp2SolrClient build() {
}
} else if (!solrUrls.isEmpty()) {
try {
- stateProvider = new Http2ClusterStateProvider(solrUrls,
httpClient);
+ if (this.internalClientBuilder != null) {
+ this.httpClient = internalClientBuilder.build();
Review Comment:
Thanks all for looking at this. I have definitly hit the "end" of my rope
on this one, just can't figure it out... My Java-fu isn't up to this one.
I am inclined to close this PR in favour of what you can whip up.. However,
since I really really really really really really want basic auth to work on
all the tools, I am hesitating to actually do that!
--
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]
