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

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

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

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: 1.5h
>  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