[ 
https://issues.apache.org/jira/browse/SOLR-17044?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jason Gerlowski updated SOLR-17044:
-----------------------------------
    Summary: Improve API path specifying in SolrJ  (was: Improve context-root 
specifying in SolrJ)

> Improve API path specifying in SolrJ
> ------------------------------------
>
>                 Key: SOLR-17044
>                 URL: https://issues.apache.org/jira/browse/SOLR-17044
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: SolrJ, v2 API
>            Reporter: Jason Gerlowski
>            Priority: Major
>
> Currently, URL-driven SolrClients receive a URL that typically ends in 
> "/solr".  Requiring users to specify this context-root in their URLs has made 
> sense historically, since all v1 paths start with "/solr".  But this has 
> become limiting as Solr's v2 APIs (which use the "/api" context root) become 
> more mature. 
> SolrJ has some string manipulation in a few spots to try to accomodate this, 
> but it's patchy and relatively brittle (relying on overly specific 
> 'instanceof' checks, etc.).  We should come up with a more reliable way to 
> ensure that SolrClient's can make both v1 and v2 requests.
> Here's one idea for how we could approach this:
> # Create a new SolrRequest method to specify the context root on a 
> per-request basis.  We could either do this directly (e.g. "getRootPath()"), 
> or indirectly (e.g. "isV2()").
> # Modify SolrClients to consider both the new method and the existing 
> "getPath" when sending requests.
> # Add logic to SolrJ to strip "/solr" from any provided base URLs, and 
> document that SolrClient URLs no longer require the "/solr" context-root 
> suffix.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to