[ 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