lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hoss Man (JIRA)" <>
Subject [jira] Commented: (LUCENE-743) IndexReader.reopen()
Date Mon, 20 Aug 2007 20:41:32 GMT


Hoss Man commented on LUCENE-743:

> I'm not sure if the code would become cleaner if we did that. Sometimes a SegmentReader
would then have to 
> return a MultiSegmentReader instance and vice versa. So we'd probably have to duplicate
some of the code in
> these two classes.

i don't hink there would be anything wrong with SegmentReader.reopen returning a MultiSegmentReader
in some cases (or vice versa) but it definitely seems wrong to me for a parent class to be
explicitly casing "this" to one of two know subclasses ... making reopen abstract in the base
class (or throw UnsupportOp if for API back compatibility) seems like the only safe way to
ensure any future IndexReader subclasses work properly.

> IndexReader.reopen()
> --------------------
>                 Key: LUCENE-743
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Index
>            Reporter: Otis Gospodnetic
>            Assignee: Michael Busch
>            Priority: Minor
>             Fix For: 2.3
>         Attachments:, lucene-743.patch, lucene-743.patch, lucene-743.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:
For additional commands, e-mail:

View raw message