You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@solr.apache.org by "Eric Pugh (Jira)" <ji...@apache.org> on 2022/09/23 16:49:00 UTC
[jira] [Commented] (SOLR-16368) Refactoring: Use SolrClient type instead of overly specific subclasses
[ https://issues.apache.org/jira/browse/SOLR-16368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17608845#comment-17608845 ]
Eric Pugh commented on SOLR-16368:
----------------------------------
So, most of the references to the HttpSolrClient in the tests are JUST to get a base url or the httpClient... If we could somehow just get that from the test infrastructure, then we would have many fewer references to HttpSolrClient.
> Refactoring: Use SolrClient type instead of overly specific subclasses
> ----------------------------------------------------------------------
>
> Key: SOLR-16368
> URL: https://issues.apache.org/jira/browse/SOLR-16368
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Reporter: David Smiley
> Assignee: Eric Pugh
> Priority: Minor
> Time Spent: 50m
> Remaining Estimate: 0h
>
> There are many references to a specific SolrClient subclass that could simply refer to SolrClient generally, or perhaps CloudSolrClient. This impedes switching/migrating to different SolrClient implementations. A similar example: we know it's a bad practice to declare a List as ArrayList even when it is one; the idea here with SolrClient is the same.
> And furthermore, often times "var solrClient = ..." (or even "var solr = ...") is totally adequate in the Solr codebase within a method because the variable name adequately communicates the type. These sorts of changes should happen first, and then weaken type references in APIs.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org