lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shai Erera (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LUCENE-3485) LuceneTaxonomyReader .decRef() may close the inner IR, renderring the LTR in a limbo.
Date Wed, 05 Oct 2011 09:23:34 GMT

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

Shai Erera commented on LUCENE-3485:
------------------------------------

I'm fine with not null-ing the caches then. It's not a big deal.

About close(), we should keep closed so that calling close() multiple times has no side-effects,
but we should change ensureReopen().
                
> 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: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message