[ 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