[
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: [email protected]
For additional commands, e-mail: [email protected]
