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

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

I'm not saying that Solr duplicated Lucene code or the other way around, I'm 
just saying that at this point, the code is duplicated. Lucene can't use Solr 
code, but Solr can use Lucene's. Making that happen would not only remove from 
Solr part of the code, but it would also improve the testing in both, Lucene 
and Solr. Using custom code also causes the need of more custom code (like in 
my previous example with the SearcherLifetimeManager). I think that as Lucene 
evolves, Solr should keep up to date with Lucene's changes and best practices, 
after all, it's the same Apache project, right? I do think these are good 
reasons. 

> Let Solr use Lucene's SeacherManager
> ------------------------------------
>
>                 Key: SOLR-5621
>                 URL: https://issues.apache.org/jira/browse/SOLR-5621
>             Project: Solr
>          Issue Type: Improvement
>    Affects Versions: 5.0
>            Reporter: Tomás Fernández Löbbe
>         Attachments: SOLR-5621.patch
>
>
> It would be nice if Solr could take advantage of Lucene's SearcherManager and 
> get rid of most of the logic related to managing Searchers in SolrCore. 
> I've been taking a look at how possible it is to achieve this, and even if I 
> haven't finish with the changes (there are some use cases that are still not 
> working exactly the same) it looks like it is possible to do.  Some things 
> still could use a lot  of improvement (like the realtime searcher management) 
> and some other not yet implemented, like Searchers on deck or 
> IndexReaderFactory
> I'm attaching an initial patch (many TODOs yet). 



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

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

Reply via email to