Re: [PR] Only apply deletion one time for unique term update in FrozenBufferedUpdates.applyTermDeletes [lucene]

2024-09-28 Thread via GitHub
github-actions[bot] commented on PR #13486: URL: https://github.com/apache/lucene/pull/13486#issuecomment-2381035818 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [PR] Introduce new encoding of BPV 21 for DocIdsWriter used in BKD Tree [lucene]

2024-09-28 Thread via GitHub
github-actions[bot] commented on PR #13521: URL: https://github.com/apache/lucene/pull/13521#issuecomment-2381035791 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [PR] Fix comment on compare suffix and target in SegmentTermsEnumFrame. [lucene]

2024-09-28 Thread via GitHub
github-actions[bot] commented on PR #13787: URL: https://github.com/apache/lucene/pull/13787#issuecomment-2381035722 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [I] TestBestCompressionLucene80DocValuesFormat fails with ArrayIndexOutOfBoundsException [lucene]

2024-09-28 Thread via GitHub
ChrisHegarty closed issue #13807: TestBestCompressionLucene80DocValuesFormat fails with ArrayIndexOutOfBoundsException URL: https://github.com/apache/lucene/issues/13807 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] TestBestSpeedLucene80DocValuesFormat fails dv for field: sorted_set has ords out of order [lucene]

2024-09-28 Thread via GitHub
ChrisHegarty closed issue #13808: TestBestSpeedLucene80DocValuesFormat fails dv for field: sorted_set has ords out of order URL: https://github.com/apache/lucene/issues/13808 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[PR] Fix bug 'DefaultPassageFormatter.format' method [lucene]

2024-09-28 Thread via GitHub
Seunghan-Jung opened a new pull request, #13832: URL: https://github.com/apache/lucene/pull/13832 ### Description In the following PR (https://github.com/apache/lucene/pull/13276), I added an option to allow customizing the sorting of Passages when using the UnifiedHighlighte

Re: [PR] First-class random access API for KnnVectorValues [lucene]

2024-09-28 Thread via GitHub
msokolov merged PR #13779: URL: https://github.com/apache/lucene/pull/13779 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.ap

Re: [PR] First-class random access API for KnnVectorValues [lucene]

2024-09-28 Thread via GitHub
msokolov commented on PR #13779: URL: https://github.com/apache/lucene/pull/13779#issuecomment-2380636241 OK, I think this is ready after a few minor issues have been addressed. I opened https://github.com/apache/lucene/issues/13831 to track replacing copy() with dictionary() -- This is

[I] Replace need for KnnVectorValues.copy() with a dictionary interface [lucene]

2024-09-28 Thread via GitHub
msokolov opened a new issue, #13831: URL: https://github.com/apache/lucene/issues/13831 ### Description The idea is to get rid of `copy()` since it does more than is needed. What's needed is an independent vector accessor over the same values source. Adrien had suggested a dictionary

Re: [PR] Drop final modifier on the public DataInput.readGroupVInts method [lucene]

2024-09-28 Thread via GitHub
uschindler commented on PR #13825: URL: https://github.com/apache/lucene/pull/13825#issuecomment-2380624362 > Thank you @rmuir, Maybe we can move `readGroupVInts()` into `GroupVIntUtil` to achieve the same goal? I raised a PR #13830 This is much better. Let's keep only the inner loop

Re: [PR] Drop final modifier on the public DataInput.readGroupVInts method [lucene]

2024-09-28 Thread via GitHub
easyice commented on PR #13825: URL: https://github.com/apache/lucene/pull/13825#issuecomment-2380612807 Thank you @rmuir, Maybe we can move `readGroupVInts()` into `GroupVIntUtil` to achieve the same goal? I raised a PR #13830 -- This is an automated message from the Apache Git Servi

[PR] Move DataInput.readGroupVInts into GroupVIntUtil [lucene]

2024-09-28 Thread via GitHub
easyice opened a new pull request, #13830: URL: https://github.com/apache/lucene/pull/13830 ### Description Discussed in #13825 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific