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

ASF subversion and git services commented on SOLR-8097:
-------------------------------------------------------

Commit 9645d4213292121d2f011f5440684ea25d7beaa3 in lucene-solr's branch 
refs/heads/branch_6_0 from Chris Hostetter
[ https://git-wip-us.apache.org/repos/asf?p=lucene-solr.git;h=9645d42 ]

SOLR-9028: Fixed some test related bugs preventing SSL + ClientAuth from ever 
being tested
(cherry picked from commit 791d1e7)

Conflicts:
        solr/core/src/test/org/apache/solr/cloud/SSLMigrationTest.java
        
solr/solrj/src/java/org/apache/solr/client/solrj/impl/HttpClientUtil.java
        solr/test-framework/src/java/org/apache/solr/SolrTestCaseJ4.java
        solr/test-framework/src/java/org/apache/solr/util/SSLTestConfig.java

Conflicts:
        
solr/core/src/test/org/apache/solr/cloud/TestMiniSolrCloudClusterSSL.java

For branch_6_0: Since SOLR-8097 will land in 6.1, remove calls in 
TestMiniSolrCloudClusterSSL to SolrTestCaseJ4.getHttpSolrClient().


> Implement a builder pattern for constructing a Solrj client
> -----------------------------------------------------------
>
>                 Key: SOLR-8097
>                 URL: https://issues.apache.org/jira/browse/SOLR-8097
>             Project: Solr
>          Issue Type: Improvement
>          Components: SolrJ
>    Affects Versions: 6.0
>            Reporter: Hrishikesh Gadre
>            Assignee: Anshum Gupta
>             Fix For: 6.0, 6.1
>
>         Attachments: SOLR-8097.patch, SOLR-8097.patch, SOLR-8097.patch, 
> SOLR-8097.patch, SOLR-8097.patch, SOLR-8097.patch, SOLR-8097.patch, 
> SOLR-8097.patch, SOLR-8097.patch, SOLR-8097.patch, SOLR-8097.patch, 
> SOLR-8097.patch, SOLR-8097.patch, SOLR-8097.patch
>
>
> Currently Solrj clients (e.g. CloudSolrClient) supports multiple constructors 
> as follows,
> public CloudSolrClient(String zkHost) 
> public CloudSolrClient(String zkHost, HttpClient httpClient) 
> public CloudSolrClient(Collection<String> zkHosts, String chroot)
> public CloudSolrClient(Collection<String> zkHosts, String chroot, HttpClient 
> httpClient)
> public CloudSolrClient(String zkHost, boolean updatesToLeaders)
> public CloudSolrClient(String zkHost, boolean updatesToLeaders, HttpClient 
> httpClient)
> It is kind of problematic while introducing an additional parameters (since 
> we need to introduce additional constructors). Instead it will be helpful to 
> provide SolrClient Builder which can provide either default values or support 
> overriding specific parameter. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to