[ 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