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

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

Commit b2e1f8ed4a309531d62125319501b313ac39605b in solr's branch 
refs/heads/main from Vladimir Vyatkin
[ https://gitbox.apache.org/repos/asf?p=solr.git;h=b2e1f8ed4a3 ]

SOLR-18130: new param "solrConnection" in solrj-streaming, support HTTP (#4320)

SOLR-18130: Universal SolrCloud connection string support (HTTP/S and ZooKeeper)

Introduced a universal Solr connection string for SolrCloud connections in 
multiple places:
streaming expressions, CLI, JDBC driver, cross-collection joins.
Connections can now be configured via HTTP(S) without requiring ZK dependencies 
and connectivity.
"zkHost" is deprecated in lieu of "solrConnection" in most places.

Notable changes:

- SolrJ: added CloudSolrClient.Builder(CloudSolrClientConnection) overload;
  renamed isZk to isZookeeper in the CloudSolrClientConnection record; added
  toString() to produce the canonical connection string
- SolrJ Streaming: added solrConnection param to all streaming functions
  (zkHost is deprecated but still supported); added withDefaultSolrConnection
  and withCollectionSolrConnection to StreamFactory; buildSolrConnection
  promoted to an instance method
- SQL JDBC driver: supports HTTP-based JDBC URLs alongside ZooKeeper-based ones
- CLI tools: added --solr-connection option; -s short flag now maps to
  --solr-connection instead of --solr-url (backwards compatible)
- Join Query Parser: added solrConnection parameter to cross-collection joins

Co-authored-by: David Smiley <[email protected]>
Co-authored-by: Eric Pugh <[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
>             Fix For: 10.1
>
>          Time Spent: 30h
>  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