[
https://issues.apache.org/jira/browse/LUCENE-3485?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13120670#comment-13120670
]
Gilad Barkai commented on LUCENE-3485:
--------------------------------------
Thanks Shai, you are right, 'closed' should only be used inside close().
About your previous comment on nullifying the caches, those are 'final' at the
moment and i'm not sure it worthwhile unfinaling them just to be nullified -
IIRC there are optimizations that may benefit from final variables, and those
caches could be hotspots.
I'll modify the ensureOpen and post a new patch.
> LuceneTaxonomyReader .decRef() may close the inner IR, renderring the LTR in
> a limbo.
> -------------------------------------------------------------------------------------
>
> Key: LUCENE-3485
> URL: https://issues.apache.org/jira/browse/LUCENE-3485
> Project: Lucene - Java
> Issue Type: Bug
> Components: modules/facet
> Affects Versions: 3.4
> Reporter: Gilad Barkai
> Assignee: Shai Erera
> Priority: Minor
> Fix For: 3.5, 4.0
>
> Attachments: LUCENE-3485.patch
>
>
> TaxonomyReader which supports ref-counting, has a decRef() method which
> delegates to an inner IndexReader and calls its .decRef(). The latter may
> close the reader (if the ref is zeroes) but the taxonomy would remain 'open'
> which will fail many of its method calls.
> Also, the LTR's .close() method does not work in the same manner as
> IndexReader's - which calls decRef(), and leaves the real closing logic to
> the decRef(). I believe this should be the right approach for the fix.
--
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]
