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

Atri Sharma commented on LUCENE-8819:
-------------------------------------

The problem is that the assert does not account for docIDs within a single 
segment i.e. when there are multiple documents within a segment collected by 
the same leaf collector. The assert should only be checking at segment 
boudaries to ensure that subsequent segments have the right sequence of DocIDs.

 

Attached is a patch to fix the same. I ran ant check and the guilty tests 
passed.

 

[~ivera] Could you check if this fix passes at your end as well?

 

[^LUCENE-8819.patch]

> org.apache.lucene.search.TestTopDocsMerge.testSort_1 failure
> ------------------------------------------------------------
>
>                 Key: LUCENE-8819
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8819
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Ignacio Vera
>            Priority: Major
>         Attachments: LUCENE-8819.patch
>
>
> It can be reproduced with:
>  
> {code:java}
> ant test  -Dtestcase=TestTopDocsMerge -Dtests.method=testSort_1 
> -Dtests.seed=E916688CE5BC9122 -Dtests.multiplier=3 -Dtests.slow=true 
> -Dtests.locale=es-US -Dtests.timezone=Pacific/Johnston -Dtests.asserts=true 
> -Dtests.file.encoding=ISO-8859-1{code}
>  
> Test fails in master and branch 8.x but it does not fail in branch 8.1. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to