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

Munendra S N edited comment on LUCENE-8103 at 3/15/20, 5:19 PM:
----------------------------------------------------------------

This also contains change were {{objectValue(doc)}} calls {{floatValue(doc)}} 
which returns default value if {{exists(doc)}} returns {{false}} but previously 
{{objectValue(doc)}} used to return {{null}} exists returned false even if 
default value is specified (in pure lucene case, there will always be default 
value). If this is expected change/fix should  we document the  change in 
behavior?

Above change resolves LUCENE-8908. Also, I wanted to share this 
[comment|https://issues.apache.org/jira/browse/LUCENE-8908?focusedCommentId=16887518&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16887518]
 which is from Solr's perspective as {{query}} could be used even without 
default value.

Just sharing my thoughts, please free to ignore if it doesn't apply here





was (Author: munendrasn):
This also contains change were {{objectValue(doc)}} calls {{floatValue(doc)}} 
which returns default value if {{exists(doc)}} returns {{false}} but previously 
{{objectValue(doc)}} used to return {{null}} exists returned false even if 
default value is specified (in pure lucene case, there will always be default 
value). If this is expected change/fix should  we document the  change is 
behavior?

This also resolves LUCENE-8908. Also, I wanted to share this 
[comment|https://issues.apache.org/jira/browse/LUCENE-8908?focusedCommentId=16887518&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16887518]
 which is from Solr's perspective as {{query}} could be used even without 
default value.

Just sharing my thoughts, please free to ignore if it doesn't apply here




> QueryValueSource should use TwoPhaseIterator
> --------------------------------------------
>
>                 Key: LUCENE-8103
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8103
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/other
>            Reporter: David Smiley
>            Assignee: David Smiley
>            Priority: Minor
>             Fix For: 8.6
>
>         Attachments: LUCENE-8103.patch, LUCENE-8103.patch
>
>          Time Spent: 3.5h
>  Remaining Estimate: 0h
>
> QueryValueSource (in "queries" module) is a ValueSource representation of a 
> Query; the score is the value.  It ought to try to use a TwoPhaseIterator 
> from the query if it can be offered. This will prevent possibly expensive 
> advancing beyond documents that we aren't interested in.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to