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

Gilad Barkai edited comment on LUCENE-3485 at 10/4/11 3:57 PM:
---------------------------------------------------------------

Thanks for the comment Shai.

The only Directory that is closed is the one that is NOT given - but rather 
wrapped around a given File.
I agree that this constructor should be removed, I did not take care of it just 
yet...

Attached a patch - Moving closer to IndexReader's ref counting paradigm, also 
introducing ensureClose().
                
      was (Author: giladbarkai):
    Moving closer to IndexReader's ref counting paradigm, also introducing 
ensureClose().
                  
> 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
>            Priority: Minor
>         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-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to