[ 
https://issues.apache.org/jira/browse/SOLR-5172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alan Woodward updated SOLR-5172:
--------------------------------

    Attachment: SOLR-5172.patch

This patch:
* starts up an embedded ZK in SolrDispatchFilter
* removes the support for ZK ensembles when running embedded ZK, simplifying 
the server management code.  If it detects that you're trying to pass an 
ensemble ZK connection string as the zkRun parameter, it prints a helpful error 
message.
* allows you to pass a ZK port as a parameter to --zkRun; it defaults to 9893 
if nothing is passed in.
* fixes ZkCLI to correctly talk to an embedded zk server if that's what is 
being run.

I'd also like to remove ZkTestServer and replace it with EmbeddedZkServer - 
will start on that next.

[[email protected]] does this look like a sensible way of doing things to 
you?

                
> Start embedded zk servers before CoreContainer loads
> ----------------------------------------------------
>
>                 Key: SOLR-5172
>                 URL: https://issues.apache.org/jira/browse/SOLR-5172
>             Project: Solr
>          Issue Type: Improvement
>            Reporter: Alan Woodward
>            Priority: Minor
>         Attachments: SOLR-5172.patch
>
>
> In SOLR-4718 we allowed solr.xml to be read from Zookeeper in cloud setups.  
> However, this doesn't work with an embedded ZK server because it doesn't 
> start until CoreContainer.load() is called, by which time solr.xml has 
> already been loaded.  We should move the embedded ZK startup/shutdown logic 
> out of CoreContainer and into SolrDispatchFilter.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to