lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Froh (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SOLR-3526) Remove classfile dependency on ZooKeeper from CoreContainer
Date Mon, 11 Jun 2012 08:39:42 GMT

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

Michael Froh commented on SOLR-3526:
------------------------------------

Oh, thanks a lot for pointing that out, Hoss! I had completely missed that part.

My wish for the removal of the KeeperException reference from CoreContainer still stands,
but using NoOpDistributingUpdateProcessorFactory lets me remove my current hacky solution
(adding a dummy org.apache.zookeeper.KeeperException in one of my libraries).
                
> 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
>            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 is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


Mime
View raw message