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

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

Commit bf610531f1c0767356ce07c35a741d42c8b786f6 in solr's branch 
refs/heads/main from vvova15
[ https://gitbox.apache.org/repos/asf?p=solr.git;h=bf610531f1c ]

SOLR-18130: New universal connection string to CloudSolrClient.Builder (#4260)

Added a universal connection string (ZK/HTTP) to CloudSolrClient.Builder

Co-authored-by: David Smiley <[email protected]>

> CloudSolrStream should not depend on ZooKeeper
> ----------------------------------------------
>
>                 Key: SOLR-18130
>                 URL: https://issues.apache.org/jira/browse/SOLR-18130
>             Project: Solr
>          Issue Type: Improvement
>          Components: SolrJ, streaming expressions
>            Reporter: David Smiley
>            Priority: Major
>              Labels: newdev, pull-request-available
>          Time Spent: 5h
>  Remaining Estimate: 0h
>
> In the SolrJ streaming expressions module, CloudSolrStream depends on 
> ZooKeeper, and thus so does its subclasses (DrillStream, TopicStream, 
> ShuffleStream, ParallelStream).  It's a runtime dependency, triggered by a 
> "zkHost" parameter that's supplied to {{SolrClientCache#getCloudSolrClient}} 
> that uses {{CloudSolrClient.Builder}} that uses reflection to try to load 
> {{ZkClientClusterStateProvider}}.
> I propose renaming these parameters in the module from zkHost to maybe 
> "quorumHosts" that could either be list of Solr HTTP (when it starts with 
> "http:" or "https:") or ZK if not.   This should bring about a new 
> well-deserved constructor to CloudSolrClient.Builder to parse a string, as 
> it's not a matter specific to streaming expressions.  This would be very 
> convenient to callers I've seen elsewhere.



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