[ 
https://issues.apache.org/jira/browse/LUCENE-8956?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Adrien Grand reopened LUCENE-8956:
----------------------------------

I had to revert this change due to test failures such as

{noformat}
13:40:13    [junit4] Suite: org.apache.lucene.search.TestQueryRescorer
13:40:13    [junit4]   2> NOTE: reproduce with: ant test  
-Dtestcase=TestQueryRescorer -Dtests.method=testRescoreIsIdempotent 
-Dtests.seed=2E252C4ACD6D510E -Dtests.slow=true -Dtests.badapples=true 
-Dtests.locale=nb -Dtests.timezone=America/Indiana/Winamac -Dtests.asserts=true 
-Dtests.file.encoding=UTF8
13:40:13    [junit4] FAILURE 0.04s J3 | 
TestQueryRescorer.testRescoreIsIdempotent <<<
13:40:13    [junit4]    > Throwable #1: junit.framework.AssertionFailedError: 
Hit 3 docnumbers don't match
13:40:13    [junit4]    > Hits length1=30       length2=30
13:40:13    [junit4]    > hit=0: doc11=0.027509231 shardIndex=-1,        
doc11=0.027509231 shardIndex=-1
13:40:13    [junit4]    > hit=1: doc59=0.026626626 shardIndex=-1,        
doc59=0.026626626 shardIndex=-1
13:40:13    [junit4]    > hit=2: doc94=0.025820786 shardIndex=-1,        
doc94=0.025820786 shardIndex=-1
13:40:13    [junit4]    > hit=3: doc14=0.025403785 shardIndex=-1,        
doc13=0.025586303 shardIndex=-1
13:40:13    [junit4]    > hit=4: doc13=0.025586303 shardIndex=-1,        
doc14=0.025403785 shardIndex=-1
13:40:13    [junit4]    > hit=5: doc69=0.02535518 shardIndex=-1,         
doc69=0.02535518 shardIndex=-1
13:40:13    [junit4]    > hit=6: doc37=0.024901403 shardIndex=-1,        
doc37=0.024901403 shardIndex=-1
13:40:13    [junit4]    > hit=7: doc73=0.023868036 shardIndex=-1,        
doc73=0.023868036 shardIndex=-1
13:40:13    [junit4]    > hit=8: doc63=0.021619176 shardIndex=-1,        
doc63=0.021619176 shardIndex=-1
13:40:13    [junit4]    > hit=9: doc72=0.019876242 shardIndex=-1,        
doc72=0.019876242 shardIndex=-1
13:40:13    [junit4]    > hit=10: doc50=0.01923588 shardIndex=-1,        
doc50=0.01923588 shardIndex=-1
13:40:13    [junit4]    > hit=11: doc10=0.018147592 shardIndex=-1,       
doc10=0.018147592 shardIndex=-1
13:40:13    [junit4]    > hit=12: doc0=0.018087288 shardIndex=-1,        
doc0=0.018087288 shardIndex=-1
13:40:13    [junit4]    > hit=13: doc98=0.017422743 shardIndex=-1,       
doc98=0.017422743 shardIndex=-1
13:40:13    [junit4]    > hit=14: doc47=0.016934035 shardIndex=-1,       
doc47=0.016934035 shardIndex=-1
13:40:13    [junit4]    > hit=15: doc6=0.016177062 shardIndex=-1,        
doc79=0.016415473 shardIndex=-1
13:40:13    [junit4]    > hit=16: doc96=0.016177062 shardIndex=-1,       
doc6=0.016177062 shardIndex=-1
13:40:13    [junit4]    > hit=17: doc79=0.016415473 shardIndex=-1,       
doc96=0.016177062 shardIndex=-1
13:40:13    [junit4]    > hit=18: doc55=0.015521079 shardIndex=-1,       
doc55=0.015521079 shardIndex=-1
13:40:13    [junit4]    > hit=19: doc34=0.011353219 shardIndex=-1,       
doc34=0.011353219 shardIndex=-1
13:40:13    [junit4]    > hit=20: doc9=0.010401427 shardIndex=-1,        
doc9=0.010401427 shardIndex=-1
13:40:13    [junit4]    > hit=21: doc12=0.0033432373 shardIndex=-1,      
doc12=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=22: doc15=0.0033432373 shardIndex=-1,      
doc15=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=23: doc23=0.0033432373 shardIndex=-1,      
doc23=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=24: doc26=0.0033432373 shardIndex=-1,      
doc26=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=25: doc31=0.0033432373 shardIndex=-1,      
doc31=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=26: doc33=0.0033432373 shardIndex=-1,      
doc33=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=27: doc41=0.0033432373 shardIndex=-1,      
doc41=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=28: doc43=0.0033432373 shardIndex=-1,      
doc43=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > hit=29: doc45=0.0033432373 shardIndex=-1,      
doc45=0.0033432373 shardIndex=-1
13:40:13    [junit4]    > for query:field:"river river"~1
13:40:13    [junit4]    >       at 
__randomizedtesting.SeedInfo.seed([2E252C4ACD6D510E:E4C37C59CD79ADE6]:0)
13:40:13    [junit4]    >       at junit.framework.Assert.fail(Assert.java:57)
13:40:13    [junit4]    >       at 
org.apache.lucene.search.CheckHits.checkEqual(CheckHits.java:205)
13:40:13    [junit4]    >       at 
org.apache.lucene.search.TestQueryRescorer.testRescoreIsIdempotent(TestQueryRescorer.java:142)
13:40:13    [junit4]    >       at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
13:40:13    [junit4]    >       at 
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
13:40:13    [junit4]    >       at 
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
13:40:13    [junit4]    >       at 
java.base/java.lang.reflect.Method.invoke(Method.java:566)
13:40:13    [junit4]    >       at 
java.base/java.lang.Thread.run(Thread.java:834)
{noformat}

[~pcsanwald] told me he is looking into it.

> QueryRescorer sort optimization
> -------------------------------
>
>                 Key: LUCENE-8956
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8956
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: core/query/scoring
>            Reporter: Paul Sanwald
>            Priority: Minor
>             Fix For: 8.3
>
>         Attachments: LUCENE-8956.patch
>
>
> This patch addresses a TODO in QueryRescorer: We should not sort the full 
> array of the results returned from rescoring, but rather only topN, when topN 
> is less than total hits.
>  
> Made this optimization with some suggestions from [~jpountz] and [~jimczi], 
> this is my first lucene patch submission.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

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

Reply via email to