[GitHub] [lucene] jpountz commented on pull request #12489: Add support for recursive graph bisection.

2023-08-20 Thread via GitHub
jpountz commented on PR #12489: URL: https://github.com/apache/lucene/pull/12489#issuecomment-1685250650 I think it's starting to look better now. I worked on some inefficiencies and applied some of the optimizations suggested by Mackenzie et al. in ["Tradeoff Options for Bipartite Graph P

[GitHub] [lucene] jpountz commented on pull request #12489: Add support for recursive graph bisection.

2023-08-20 Thread via GitHub
jpountz commented on PR #12489: URL: https://github.com/apache/lucene/pull/12489#issuecomment-1685382263 I ran the benchmark multiple times to see if the slowdown on `OrHighLow` reproduced, and it does. I took the first `OrHighLow` query in the tasks file: `OrHighLow: 2005 valois # freq=835

[GitHub] [lucene] Yuti-G commented on issue #12418: Reproducible TestDrillSideways failure

2023-08-20 Thread via GitHub
Yuti-G commented on issue #12418: URL: https://github.com/apache/lucene/issues/12418#issuecomment-1685632025 Hello Adrien, Thanks for raising this issue! I was trying to locate which release version / commit that broke this test in the beginning and I found that the test starts pass

[GitHub] [lucene] iverase commented on a diff in pull request #12460: Allow reading binary doc values as a DataInput

2023-08-20 Thread via GitHub
iverase commented on code in PR #12460: URL: https://github.com/apache/lucene/pull/12460#discussion_r1299684458 ## lucene/core/src/java/org/apache/lucene/codecs/lucene90/Lucene90DocValuesProducer.java: ## @@ -820,6 +822,213 @@ public BytesRef binaryValue() throws IOException {