[
https://issues.apache.org/jira/browse/LUCENE-3558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13147084#comment-13147084
]
Simon Willnauer commented on LUCENE-3558:
-----------------------------------------
bq. Can we also move SearcherLifetimeManager
I wasn't sure about this one - SeracherManager and NRTManager went through
serious refactorings since they where added. But we can still do that if it is
in core... I will mover the NRTCachingDirectory in a different issue.
bq. I'm curious why you used svn copy instead of svn move?
I used svn cp since the patch removes the files and otherwise some systems have
problems if the files don't exist anymore. I used svn mv to create the patch
actually :)
> SearcherManager and NRTManager should be in the same package
> ------------------------------------------------------------
>
> Key: LUCENE-3558
> URL: https://issues.apache.org/jira/browse/LUCENE-3558
> Project: Lucene - Java
> Issue Type: Improvement
> Reporter: Robert Muir
> Assignee: Simon Willnauer
> Fix For: 3.5, 4.0
>
> Attachments: LUCENE-3558.patch, LUCENE-3558.sh
>
>
> I didnt even know NRTManager was still around, because its in the .index
> package, whereas SearcherManager is in the .search package.
> Separately, I don't like that this stuff is so 'hard' with core lucene...
> would it be so bad if this stuff was added to core?
> I suspect a lot of people have issues with this stuff (see
> http://www.lucidimagination.com/search/document/37964e5f0e5d733b) for example.
> Worst case is just that, combine mistakes with trying to manage this stuff
> with MMap unmapping and total lack of error detection
> for searching closed readers (LUCENE-3439) and its a mess.
--
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: [email protected]
For additional commands, e-mail: [email protected]
