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

Mark Miller commented on SOLR-1431:
-----------------------------------

bq. I think it could conflict with other uses of Zookeeper when the library 
versions are different.

Yeah - always a problem with dependencies like this. It's hard to say what 
direction we go right now though - some have argued even non zookeeper mode 
should be single install zookeeper mode instead. Has it's advantages and 
disadvantages I think. For me, I can really only take it an issue at a team, 
and while I hope to drive some more things around SolrCloud soon, it's 
obviously been a while. Others have some issues open, but more ideas are always 
good.

I certainly agree that CoreContainer could be modularized better - would help 
for testing too. I have an issue to do this for the persistence code (baby 
steps :) ), but feel free to open further issues.

I somewhat took the easy route in integrating zookeeper - there are certainly 
lots of improvements that could be made overall. And TODO's to finish - I think 
a couple guys have done a few from the wiki in various issues, and I know 
loggly has privately impl'd a couple from their talk at revolution (would be 
cool to see that come back, but I know they are busy guys). I love TODO's - 
minimal effort, but when you put one at a future pain point, your code doesn't 
look so stupid even when it's not perfect yet :)

We should discuss in other issues though.

> CommComponent abstracted
> ------------------------
>
>                 Key: SOLR-1431
>                 URL: https://issues.apache.org/jira/browse/SOLR-1431
>             Project: Solr
>          Issue Type: Improvement
>          Components: search
>    Affects Versions: 4.0
>            Reporter: Jason Rutherglen
>            Assignee: Noble Paul
>             Fix For: 4.0
>
>         Attachments: SOLR-1431.patch, SOLR-1431.patch, SOLR-1431.patch, 
> SOLR-1431.patch, SOLR-1431.patch, SOLR-1431.patch, SOLR-1431.patch, 
> SOLR-1431.patch, SOLR-1431.patch, SOLR-1431.patch, SOLR-1431.patch
>
>
> We'll abstract CommComponent in this issue.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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

Reply via email to