Eric Pugh

Results 315 comments of 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...

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...