[
https://issues.apache.org/jira/browse/LUCENE-6371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14564740#comment-14564740
]
Robert Muir commented on LUCENE-6371:
-------------------------------------
{quote}
There's still an extra visit in SpanMTQWrapper, but I think we can fix that by
adding a SpanTermQuery(Term, TermContext) constructor, like we have with the
standard TermQuery. Maybe we should carry this over to LUCENE-6466, as here
it's getting mixed up with the span collection API, which is a separate thing.
I'll put up a patch.
{quote}
OK, this sounds good to me, because it would at least be consistent with
TermQuery.
{quote}
Maybe we should carry this over to LUCENE-6466, as here it's getting mixed up
with the span collection API, which is a separate thing. I'll put up a patch.
{quote}
OK I agree, lets not try to tackle it all at once in one patch. Lets just fix
trunk until we are happy on whatever issues we need. Then we backport
everything to 5.x for 5.3 here.
I have the feeling its really not that far away (your cleanups already here
addressed a lot of my concerns), but it would be good to make sure the api
changes support what we need. Fixing this MTQ termcontext stuff would a great
improvement.
Just as a reminder I am still concerned about LUCENE-6495, which is test
failures introduced from LUCENE-6466 (somehow scoring is changed, and only when
using java 7!!!). This makes life tricky because trunk requires java 8 so we
can't easily dig in. But maybe we can just do a lot of beasting before
backporting the whole thing and try to figure that one out at that time.
> Improve Spans payload collection
> --------------------------------
>
> Key: LUCENE-6371
> URL: https://issues.apache.org/jira/browse/LUCENE-6371
> Project: Lucene - Core
> Issue Type: Improvement
> Reporter: Paul Elschot
> Assignee: Alan Woodward
> Priority: Minor
> Fix For: Trunk, 5.3
>
> Attachments: LUCENE-6371.patch, LUCENE-6371.patch, LUCENE-6371.patch,
> LUCENE-6371.patch
>
>
> Spin off from LUCENE-6308, see the comments there from around 23 March 2015.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
