[
https://issues.apache.org/jira/browse/LUCENE-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12742045#action_12742045
]
Mark Miller commented on LUCENE-533:
------------------------------------
Paul:
Spans is breaking back compat in this release. This is the opportunity to
change anything.
Do you think we should add something like this? Do you think it might make
sense to make Spans abstract?
Its an interface thats gaining methods this release, so back compat is gone
anyway (Spans back compat was lost in the last release, and we are correcting
things to a degree in 2.9). If we can help pave the way for the future in a
reasonably small amount of time - this is likely the best opportunity.
> SpanQuery scoring: SpanWeight lacks a recursive traversal of the query tree
> ---------------------------------------------------------------------------
>
> Key: LUCENE-533
> URL: https://issues.apache.org/jira/browse/LUCENE-533
> Project: Lucene - Java
> Issue Type: Bug
> Components: Search
> Affects Versions: 1.9
> Reporter: Vincent Le Maout
> Priority: Minor
>
> I found the computing of weights to be somewhat different according to the
> query type (BooleanQuery versus SpanQuery) :
> org.apache.lucene.search.BooleanQuery.BooleanWeight :
> public BooleanWeight(Searcher searcher)
> throws IOException {
> this.similarity = getSimilarity(searcher);
> for (int i = 0 ; i < clauses.size(); i++) {
> BooleanClause c = (BooleanClause)clauses.elementAt(i);
> weights.add(c.getQuery().createWeight(searcher));
> }
> }
> which looks like a recursive descent through the tree, taking into account
> the weights of all the nodes, whereas :
> org.apache.lucene.search.spans.SpanWeight :
> public SpanWeight(SpanQuery query, Searcher searcher)
> throws IOException {
> this.similarity = query.getSimilarity(searcher);
> this.query = query;
> this.terms = query.getTerms();
> idf = this.query.getSimilarity(searcher).idf(terms, searcher);
> }
> lacks any traversal and according to what I have understood so far from the
> rest
> of the code, only takes into account the boost of the tree root in
> SumOfSquareWeights(),
> which is consistent with the resulting scores not considering the boost of
> the tree
> leaves.
> vintz
--
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
