[ https://issues.apache.org/jira/browse/SOLR-17256?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17852212#comment-17852212 ]
ASF subversion and git services commented on SOLR-17256: -------------------------------------------------------- Commit 2cf82e59a85a00bae5c9fbb44dd8942364b2105a in solr's branch refs/heads/branch_9x from David Smiley [ https://gitbox.apache.org/repos/asf?p=solr.git;h=2cf82e59a85 ] SOLR-17256: Deprecate SolrRequest get/set BasePath and RequestWriter.getPath (#2494) Just deprecation. Later, will follow-up with replacements and removal, including CHANGES.txt. (cherry picked from commit dfc8b587ea983759fd65061e7e1601555132ae3b) > Remove SolrRequest.getBasePath setBasePath > ------------------------------------------ > > Key: SOLR-17256 > URL: https://issues.apache.org/jira/browse/SOLR-17256 > Project: Solr > Issue Type: Improvement > Components: SolrJ > Reporter: David Smiley > Priority: Minor > Labels: newdev > Time Spent: 20m > Remaining Estimate: 0h > > SolrRequest has a getBasePath & setBasePath. The naming is poor; it's the > URL base to the Solr node like "http://localhost:8983/solr". It's only > recognized by HttpSolrClient; LBSolrClient (used by CloudSolrClient) ignores > it and will in fact mutate the passed in request to its liking, which is > rather ugly because it means a request cannot be used concurrently if the > user wants to. But moreover I think there's a conceptual discordance of > placing this concept on SolrRequest given that some clients want to route > requests to nodes *they* choose. I propose removing this from SolrRequest > and instead adding a method specific to HttpSolrClient. Almost all existing > usages of setBasePath immediately execute the request on an HttpSolrClient, > so should be easy to change. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org