epugh commented on PR #1012: URL: https://github.com/apache/solr/pull/1012#issuecomment-1276146141
I went ahead and looked at the other specific subclasses of `SolrClient` and `SolrCloudClient`... I think at THIS POINT, I am now done with this PR. I am running all the tests again, and assuming they pass and a I get LGTM, then I'll merge. I will start a thread on Dev about other refactoring/cleanup areas, maybe `newdev` candidates? -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
