lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Uwe Schindler (JIRA)" <>
Subject [jira] Commented: (LUCENE-2694) MTQ rewrite + weight/scorer init should be single pass
Date Sat, 18 Dec 2010 10:36:02 GMT


Uwe Schindler commented on LUCENE-2694:

I have also some things:
- We currently don't support seeking a FilteredTermsEnum, this is disallowed by UnsupportedOperationException
(we may change this, but its complicated, Robert and me are thinking about it, but for now
its disallowed, as it would break the enum logic). So the TermState seek method in FilteredTermsEnum
should also throw UOE:
/** This enum does not support seeking!
 * @throws UnsupportedOperationException
public SeekStatus seek(BytesRef term, boolean useCache) throws IOException {
  throw new UnsupportedOperationException(getClass().getName()+" does not support seeking");
- For what is setNextReader in TermCollector? I don't like that, but you seems to need it
for the PerReaderTermState. The collector should really only work on the enum not on any reader.

Thats what I have seen on first patch review, will now apply patch and look closer into it
:-) But the first point is important, FilteredTermsEnum currently should not support seeking.

> MTQ rewrite + weight/scorer init should be single pass
> ------------------------------------------------------
>                 Key: LUCENE-2694
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Search
>            Reporter: Michael McCandless
>            Assignee: Michael McCandless
>             Fix For: 4.0
>         Attachments: LUCENE-2694.patch, LUCENE-2694.patch, LUCENE-2694.patch, LUCENE-2694.patch
> Spinoff of LUCENE-2690 (see the hacked patch on that issue)...
> Once we fix MTQ rewrite to be per-segment, we should take it further and make weight/scorer
init also run in the same single pass as rewrite.

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