lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael McCandless (Updated) (JIRA)" <>
Subject [jira] [Updated] (LUCENE-3769) Simplify NRTManager
Date Sat, 11 Feb 2012 21:34:59 GMT


Michael McCandless updated LUCENE-3769:

    Attachment: LUCENE-3769.patch

New patch, exposing the private SearcherManager via

I really don't like the trappiness, ie that an app will mistakenly
call SM.maybeReopen instead of NRTM.maybeReopen, thus silently causing
threads to hang....  But I think once LUCENE-3761 is in we can fix
this... I'll open a separate issue to address this....

I think this is ready!
> Simplify NRTManager
> -------------------
>                 Key: LUCENE-3769
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/search
>            Reporter: Michael McCandless
>            Assignee: Michael McCandless
>             Fix For: 3.6, 4.0
>         Attachments: LUCENE-3769.patch, LUCENE-3769.patch
> NRTManager is hairy now, because the applyDeletes is separately passed
> to ctor, passed to maybeReopen, passed to getSearcherManager, etc.
> I think, instead, you should pass it only to the ctor, and if you have
> some cases needing deletes and others not then you can make two
> NRTManagers.  This should be no less efficient than we have today,
> just simpler.
> I think it will also enable NRTManager to subclass ThingyManager
> (LUCENE-3761).

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message