[
https://issues.apache.org/jira/browse/LUCENE-1001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12544532
]
Doug Cutting commented on LUCENE-1001:
--------------------------------------
> how do I get access to the position payloads in the order that they occur in
> the PQ?
Why do you need them in that order? In the API you propose in the description
of this issue, there's no clear association between the payloads returned and
the query terms. So I don't yet see how the order of the payloads is useful.
You could pretty easily return the list of payloads along with their positions
by iterating through the list of sub-queries. The problem is that providing
that sorted by position is expensive. Perhaps you could leave any such
sorting, if required, to the application?
> 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
>
> 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}
--
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
