Eric Pugh
Eric Pugh
@cpoerschke I wanted to get this sorted, any chance you could be another set of eyes????
@cpoerschke @tboeghk want to pick this one up again....? (Though I know @tboeghk is out on vacation till September!)....
Thoughts @cpoerschke and @danrosher on this being ready for merging? Seems like a valuable bug fix! Or is there still a lack of clarity on the right behavior?
At the risk of bikeshedding.... How about `ReplicaStateProps.LEADER` instead of `ReplicaSProps.LEADER` since we don't have an existing concept of `SProps` throughout our codebase?
Looks like the content about `CloudSolrClient` in the ref guide (https://solr.apache.org/guide/solr/latest/deployment-guide/solrj.html#building-and-running-solrj-applications) refers to the use of the solr url for confiugration I wonder if we should tweak the text thgouh...
Really looking forward to this!
Docs look good!
Docs are looking nice!
I sent a note to dev mailing list to see if @joel-bernstein can review it ;-) If he can't, I can try, but it's just an area I haven't worked...