[ 
https://issues.apache.org/jira/browse/SOLR-10455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17654057#comment-17654057
 ] 

Jason Gerlowski commented on SOLR-10455:
----------------------------------------

Great work [~epugh]!  One question/suggestion:

The Javadocs on the (now deprecated) {{setParser}} method pointed to where 
users could go on {{SolrRequest}} if they needed to override the default 
ResponseParser for a particular request.  AFAICT, that detail didn't make it 
into the javadocs that you added for {{SolrClientBuilder.withResponseParser}}.  
Was there a particular reason for this, or was it just oversight?

IMO it's an important detail to keep around.  Solr has some APIs that only 
support a subset of response formats, and I'd hate to see unaware users create 
new clients just to handle these cases.

I'd be happy to take a crack at updating those javadocs, assuming the omission 
was unintentional?  Let me know!

> setParser should be deprecated in favor of SolrClientBuilder methods
> --------------------------------------------------------------------
>
>                 Key: SOLR-10455
>                 URL: https://issues.apache.org/jira/browse/SOLR-10455
>             Project: Solr
>          Issue Type: Sub-task
>          Components: SolrJ
>            Reporter: Jason Gerlowski
>            Assignee: Eric Pugh
>            Priority: Minor
>             Fix For: 7.0, 9.2
>
>
> Now that builders are in place for {{SolrClients}}, the setters used in each 
> {{SolrClient}} can be deprecated, and their functionality moved over to the 
> Builders. This change brings a few benefits:
> - unifies {{SolrClient}} configuration under the new Builders. It'll be nice 
> to have all the knobs, and levers used to tweak {{SolrClient}}s available in 
> a single place (the Builders).
> - reduces {{SolrClient}} thread-safety concerns. Currently, clients are 
> mutable. Using some {{SolrClient}} setters can result in erratic and "trappy" 
> behavior when the clients are used across multiple threads.
> This subtask endeavors to change this behavior for the {{setParser}} setter 
> on all {{SolrClient}} implementations.



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

Reply via email to