You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Tomás Fernández Löbbe (JIRA)" <ji...@apache.org> on 2015/12/18 20:24:46 UTC

[jira] [Commented] (SOLR-3526) Remove classfile dependency on ZooKeeper from CoreContainer

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

Tomás Fernández Löbbe commented on SOLR-3526:
---------------------------------------------

I think the change makes sense, it's a valid concern that people using EmbeddedSolrServer doesn't want ZooKeeper dependencies. 
I'm wondering if there is an easy way to test this so that dependencies are not inadvertently added later.

> Remove classfile dependency on ZooKeeper from CoreContainer
> -----------------------------------------------------------
>
>                 Key: SOLR-3526
>                 URL: https://issues.apache.org/jira/browse/SOLR-3526
>             Project: Solr
>          Issue Type: Wish
>          Components: SolrCloud
>    Affects Versions: 4.0-ALPHA
>            Reporter: Michael Froh
>
> We are using Solr as a library embedded within an existing application, and are currently developing toward using 4.0 when it is released.
> We are currently instantiating SolrCores with null CoreDescriptors (and hence no CoreContainer), since we don't need SolrCloud functionality (and do not want to depend on ZooKeeper).
> A couple of months ago, SearchHandler was modified to try to retrieve a ShardHandlerFactory from the CoreContainer. I was able to work around this by specifying a dummy ShardHandlerFactory in the config.
> Now UpdateRequestProcessorChain is inserting a DistributedUpdateProcessor into my chains, again triggering a NPE when trying to dereference the CoreDescriptor.
> I would happily place the SolrCores in CoreContainers, except that CoreContainer imports and references org.apache.zookeeper.KeeperException, which we do not have (and do not want) in our classpath. Therefore, I get a ClassNotFoundException when loading the CoreContainer class.
> Ideally (IMHO), ZkController should isolate the ZooKeeper dependency, and simply rethrow KeeperExceptions as org.apache.solr.common.cloud.ZooKeeperException (or some Solr-hosted checked exception). Then CoreContainer could remove the offending import/references.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org