[
https://issues.apache.org/jira/browse/LUCENE-1327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Michael Busch resolved LUCENE-1327.
-----------------------------------
Resolution: Fixed
Committed revision 756669.
> TermSpans skipTo() doesn't always move forwards
> -----------------------------------------------
>
> Key: LUCENE-1327
> URL: https://issues.apache.org/jira/browse/LUCENE-1327
> Project: Lucene - Java
> Issue Type: Bug
> Components: Query/Scoring, Search
> Affects Versions: 1.9, 2.0.0, 2.1, 2.2, 2.3, 2.3.1, 2.3.2, 2.4
> Reporter: Moti Nisenson
> Assignee: Michael Busch
> Priority: Minor
> Fix For: 2.9
>
> Attachments: lucene-1327.patch
>
>
> In TermSpans (or the anonymous Spans class returned by SpansTermQuery,
> depending on the version), the skipTo() method is improperly implemented if
> the target doc is less than or equal to the current doc:
> public boolean skipTo(int target) throws IOException {
> // are we already at the correct position?
> if (doc >= target) {
> return true;
> }
> ...
> This violates the correct behavior (as described in the Spans interface
> documentation), that skipTo() should always move forwards, in other words the
> correct implementation would be:
> if (doc >= target) {
> return next();
> }
> This bug causes particular problems if one wants to use the payloads feature
> - this is because if one loads a payload, then performs a skipTo() to the
> same document, then tries to load the "next" payload, the spans hasn't
> changed position and it attempts to load the same payload again (which is an
> error).
--
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-unsubscr...@lucene.apache.org
For additional commands, e-mail: java-dev-h...@lucene.apache.org
