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

ASF subversion and git services commented on LUCENE-10384:
----------------------------------------------------------

Commit f56140335a0cdb3bff2e5eeb6ff0e3c5bafc3666 in lucene's branch 
refs/heads/branch_9x from Mayya Sharipova
[ https://gitbox.apache.org/repos/asf?p=lucene.git;h=f561403 ]

LUCENE-10384: Simplify LongHeap small addition (#623)

LUCENE-10384 and PR#615 introduced encoding f into NeighborQueue.
But one function `nodes()` was remained to add this encoding.

Also modify the test that would fail without this patch.

> Simplify LongHeap
> -----------------
>
>                 Key: LUCENE-10384
>                 URL: https://issues.apache.org/jira/browse/LUCENE-10384
>             Project: Lucene - Core
>          Issue Type: Task
>            Reporter: Adrien Grand
>            Priority: Minor
>             Fix For: 9.1
>
>          Time Spent: 2h 40m
>  Remaining Estimate: 0h
>
> LongHeap makes its order configurable, but the only place where we use max 
> heaps, we also encode values anyway. So let's make LongHeap always a min-heap 
> and update the encoding so that the min-heap would effectively act as a max 
> heap when necessary?



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

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

Reply via email to