[
https://issues.apache.org/jira/browse/SOLR-18130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18066231#comment-18066231
]
Vladimir Vyatkin commented on SOLR-18130:
-----------------------------------------
Hello! I am new in solr, can i pick this ticket?
It looks like we need to get rid of places in the "solrj-streaming" module
where the use of ZookeeperCSP is hardcoded in favor of an abstraction from a
concrete CSP implementation and a neutral name for quorum variables/properties.
And also correct the points where solrj-streaming is used, and do tests to make
sure that it works equally with both HTTP and Zk? Do I understand correctly?
> 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
>
> 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]
