lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Busch (JIRA)" <j...@apache.org>
Subject [jira] Updated: (LUCENE-743) IndexReader.reopen()
Date Wed, 31 Oct 2007 07:03:50 GMT

     [ https://issues.apache.org/jira/browse/LUCENE-743?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Michael Busch updated LUCENE-743:
---------------------------------

    Attachment: lucene-743-take4.patch

{quote}
Patch looks great!  I'm still working through it but found a few small
issues...
{quote}
Thanks Mike! Very good review and feedback!

{quote}
It might be good to put a "assert refCount > 0" at various places like...
{quote}
Agreed. I added those asserts to incRef() and decRef(). I didn't add it
to ensureOpen(), because it throws an AlreadyClosedException anyway, and
some testcases check if this exception is thrown.

{quote}
I'm seeing a failure in contrib/memory testcase:
{quote}
Oups, I fixed this already. I changed the (deprecated) ctr 
IndexReader.IndexReader(Directory) to call this() which sets the refCount 
to 1. The test passes then. I made this fix yesterday, I think I just 
forgot to update the patch file before I submitted it, sorry about this.

{quote}
  - In multiple places you catch an IOException and undo the attempted
    re-open, but shouldn't this be a try/finally instead so you also
    clean up on hitting any unchecked exceptions?
{quote}
Yes of course! Changed it.

{quote}
  - I think you need an explicit refCount for the Norm class in
    SegmentReader.
{quote}
OK I see. I made this change as well. I also made the change that
there is no chain, but one starting SegmentReader which all re-opened 
ones reference (see below). Now this starting SegmentReader won't close 
its norms until all other readers that reference it are closed (RC=0),
because only then doClose() is called, which calls closeNorms().
Do you see an easy way how to improve this?
Hmm, probably I have to definalize IndexReader.incRef() and decRef()
and overwrite them in SegmentReader. Then SegmentReader.incRef() would
also incRef the norms, SegmentReader.decref() would decref the norms,
and somehow a clone that shares references the reader but not the norms
(because they changed) would only incref the reader itself, but not
the norms... Or do you see an easier way?

{quote}
  - If you have a long series of reopens, then, all SegmentReaders in
    the chain will remain alive.  So this is a [small] memory leak
    with time.  I think if you changed referencedSegmentReader to
    always be the *starting* SegmentReader then this chain is broken
{quote}
Good point. Ok I changed this and also the test cases that check the refCount
values.

> IndexReader.reopen()
> --------------------
>
>                 Key: LUCENE-743
>                 URL: https://issues.apache.org/jira/browse/LUCENE-743
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Index
>            Reporter: Otis Gospodnetic
>            Assignee: Michael Busch
>            Priority: Minor
>             Fix For: 2.3
>
>         Attachments: IndexReaderUtils.java, lucene-743-take2.patch, lucene-743-take3.patch,
lucene-743-take4.patch, lucene-743.patch, lucene-743.patch, lucene-743.patch, MyMultiReader.java,
MySegmentReader.java, varient-no-isCloneSupported.BROKEN.patch
>
>
> This is Robert Engels' implementation of IndexReader.reopen() functionality, as a set
of 3 new classes (this was easier for him to implement, but should probably be folded into
the core, if this looks good).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: java-dev-help@lucene.apache.org


Mime
View raw message