Could you turn that code into a simple test...or submit the code so I
could reduce it to a test?
Jonathan Mamou (JIRA) wrote:
[ https://issues.apache.org/jira/browse/LUCENE-1001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12647978#action_12647978 ]
Jonathan Mamou commented on LUCENE-1001:
----------------------------------------
In my document, I have ".... term1 ... term1 term2 ..."
offset of term1: 10, 20
offset of term2: 21
When running the query SpanNearQuery(term1 term2),
according to span.start, I get 20
while I get 10 and 21 when reading the payloads of the span.
I would expect to get 20 and 21when reading the payloads of the span.
Add Payload retrieval to Spans
------------------------------
Key: LUCENE-1001
URL: https://issues.apache.org/jira/browse/LUCENE-1001
Project: Lucene - Java
Issue Type: New Feature
Components: Search
Reporter: Grant Ingersoll
Assignee: Grant Ingersoll
Priority: Minor
Fix For: 2.4
Attachments: LUCENE-1001.patch, LUCENE-1001.patch, LUCENE-1001.patch,
LUCENE-1001.patch, LUCENE-1001.patch, LUCENE-1001.patch, LUCENE-1001.patch,
LUCENE-1001.patch
It will be nice to have access to payloads when doing SpanQuerys.
See http://www.gossamer-threads.com/lists/lucene/java-dev/52270 and
http://www.gossamer-threads.com/lists/lucene/java-dev/51134
Current API, added to Spans.java is below. I will try to post a patch as soon
as I can figure out how to make it work for unordered spans (I believe I have
all the other cases working).
{noformat}
/**
* Returns the payload data for the current span.
* This is invalid until [EMAIL PROTECTED] #next()} is called for
* the first time.
* This method must not be called more than once after each call
* of [EMAIL PROTECTED] #next()}. However, payloads are loaded lazily,
* so if the payload data for the current position is not needed,
* this method may not be called at all for performance reasons.<br>
* <br>
* <p><font color="#FF0000">
* WARNING: The status of the <b>Payloads</b> feature is experimental.
* The APIs introduced here might change in the future and will not be
* supported anymore in such a case.</font>
*
* @return a List of byte arrays containing the data of this payload
* @throws IOException
*/
// TODO: Remove warning after API has been finalized
List/*<byte[]>*/ getPayload() throws IOException;
/**
* Checks if a payload can be loaded at this position.
* <p/>
* Payloads can only be loaded once per call to
* [EMAIL PROTECTED] #next()}.
* <p/>
* <p><font color="#FF0000">
* WARNING: The status of the <b>Payloads</b> feature is experimental.
* The APIs introduced here might change in the future and will not be
* supported anymore in such a case.</font>
*
* @return true if there is a payload available at this position that can be
loaded
*/
// TODO: Remove warning after API has been finalized
public boolean isPayloadAvailable();
{noformat}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
