lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Miller <markrmil...@gmail.com>
Subject Re: [jira] Commented: (LUCENE-1001) Add Payload retrieval to Spans
Date Sun, 16 Nov 2008 14:30:08 GMT
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 {@link #next()} is called for
>>    * the first time.
>>    * This method must not be called more than once after each call
>>    * of {@link #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
>>    * {@link #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: java-dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: java-dev-help@lucene.apache.org


Mime
View raw message