[ 
https://issues.apache.org/jira/browse/LUCENE-8099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16293837#comment-16293837
 ] 

Alan Woodward commented on LUCENE-8099:
---------------------------------------

Thanks for reviewing David

bq. Here you've made the queries module now depend on the expressions module

This is a test dependency only, so it won't affect any downstream modules.

bq. org.apache.lucene.search.uhighlight.MultiTermHighlighting

Will update.  And yes, it would make life much easier if we could recurse query 
trees...

bq. Is it heavyweight to suggest using the expressions module just to multiply 
a score?

DoubleValuesSource had a helper to create sources that used arbitrary functions 
when I first committed it, but I removed it a bit later because it messed up 
equality tests (essentially, you can't check Java closures for equality).  I 
think expressions are a good solution?  They allow for hashcode/equals tests, 
they have a very simple syntax, and they're completely general.  My worry about 
adding a special score-multiplying source is that then you start to add other 
specialised sources, and we end up with the existential horror that is the 
queries.function.valuesource package.  We could maybe make SimpleBindings more 
user friendly in another issue if you think it's still complex.

bq. Maybe BoostQuery could use a bit of Javadoc

Good idea

> Deprecate CustomScoreQuery, BoostedQuery and BoostingQuery
> ----------------------------------------------------------
>
>                 Key: LUCENE-8099
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8099
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Alan Woodward
>            Assignee: Alan Woodward
>            Priority: Minor
>         Attachments: LUCENE-8099.patch
>
>
> After LUCENE-7998, these three queries can all be replaced by a 
> FunctionScoreQuery.  Using lucene-expressions makes them much easier to use 
> as well.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to