[PR] Add WrappingReuseStrategy for AnalyzerWrapper [lucene]

2025-01-20 Thread via GitHub
mayya-sharipova opened a new pull request, #14154: URL: https://github.com/apache/lucene/pull/14154 Add a new reuse strategy WrappingReuseStrategy that consults he wrapped analyzer's strategy to decide if components can be reused or need to be updated. Add an optional construct

Re: [I] TestBPReorderingMergePolicy fails CheckIndex.testHnswGraph [lucene]

2025-01-20 Thread via GitHub
msokolov commented on issue #14127: URL: https://github.com/apache/lucene/issues/14127#issuecomment-2603066263 I spent some time trying to understand how this arose, and working on a fix, and I believe that the BP reordering exposed a pre-existing behavior in the component-merging code whic

Re: [PR] Integrating GPU based Vector Search using cuVS [lucene]

2025-01-20 Thread via GitHub
uschindler commented on code in PR #14131: URL: https://github.com/apache/lucene/pull/14131#discussion_r1922705455 ## lucene/sandbox/src/java/module-info.java: ## @@ -20,6 +20,9 @@ requires org.apache.lucene.core; requires org.apache.lucene.queries; requires org.apache.

Re: [PR] Integrating GPU based Vector Search using cuVS [lucene]

2025-01-20 Thread via GitHub
uschindler commented on code in PR #14131: URL: https://github.com/apache/lucene/pull/14131#discussion_r1922703919 ## lucene/sandbox/src/java/module-info.java: ## @@ -20,6 +20,9 @@ requires org.apache.lucene.core; requires org.apache.lucene.queries; requires org.apache.

Re: [I] TestBPReorderingMergePolicy fails CheckIndex.testHnswGraph [lucene]

2025-01-20 Thread via GitHub
iverase commented on issue #14127: URL: https://github.com/apache/lucene/issues/14127#issuecomment-2602937439 In case useful, I could reduce the test to something like: ``` public void testFastCommits() throws IOException { IndexWriterConfig config = newIndexWriterConfig()

Re: [PR] Integrating GPU based Vector Search using cuVS [lucene]

2025-01-20 Thread via GitHub
ChrisHegarty commented on code in PR #14131: URL: https://github.com/apache/lucene/pull/14131#discussion_r1922626291 ## lucene/sandbox/src/java/module-info.java: ## @@ -20,6 +20,9 @@ requires org.apache.lucene.core; requires org.apache.lucene.queries; requires org.apach

Re: [PR] Integrating GPU based Vector Search using cuVS [lucene]

2025-01-20 Thread via GitHub
uschindler commented on code in PR #14131: URL: https://github.com/apache/lucene/pull/14131#discussion_r1922618477 ## lucene/sandbox/src/java/module-info.java: ## @@ -20,6 +20,9 @@ requires org.apache.lucene.core; requires org.apache.lucene.queries; requires org.apache.

Re: [PR] Integrating GPU based Vector Search using cuVS [lucene]

2025-01-20 Thread via GitHub
uschindler commented on code in PR #14131: URL: https://github.com/apache/lucene/pull/14131#discussion_r1922618477 ## lucene/sandbox/src/java/module-info.java: ## @@ -20,6 +20,9 @@ requires org.apache.lucene.core; requires org.apache.lucene.queries; requires org.apache.

Re: [PR] Integrating GPU based Vector Search using cuVS [lucene]

2025-01-20 Thread via GitHub
ChrisHegarty commented on code in PR #14131: URL: https://github.com/apache/lucene/pull/14131#discussion_r1922550039 ## lucene/sandbox/src/java/module-info.java: ## @@ -20,6 +20,9 @@ requires org.apache.lucene.core; requires org.apache.lucene.queries; requires org.apach

Re: [PR] Add AbstractKnnVectorQuery.seed for seeded HNSW [lucene]

2025-01-20 Thread via GitHub
seanmacavaney closed pull request #13635: Add AbstractKnnVectorQuery.seed for seeded HNSW URL: https://github.com/apache/lucene/pull/13635 -- 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 specifi

Re: [PR] Add AbstractKnnVectorQuery.seed for seeded HNSW [lucene]

2025-01-20 Thread via GitHub
seanmacavaney commented on PR #13635: URL: https://github.com/apache/lucene/pull/13635#issuecomment-2602464296 closing due to #14084 -- 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 c

Re: [I] Seeding HNSW Search [lucene]

2025-01-20 Thread via GitHub
seanmacavaney commented on issue #13634: URL: https://github.com/apache/lucene/issues/13634#issuecomment-2602462598 Thanks @benwtrent! -- 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

Re: [I] Seeding HNSW Search [lucene]

2025-01-20 Thread via GitHub
benwtrent closed issue #13634: Seeding HNSW Search URL: https://github.com/apache/lucene/issues/13634 -- 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: is

Re: [I] Seeding HNSW Search [lucene]

2025-01-20 Thread via GitHub
benwtrent commented on issue #13634: URL: https://github.com/apache/lucene/issues/13634#issuecomment-2602459537 This has now been merged! Huzzah! -- 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 t

Re: [I] Add an optional bandwidth cap to `TieredMergePolicy`? [lucene]

2025-01-20 Thread via GitHub
jpountz commented on issue #14148: URL: https://github.com/apache/lucene/issues/14148#issuecomment-2602033893 I wonder if this is something that could be implemented in the merge scheduler rather than in the merge policy. Thinking out loud: the merge policy's responsibility is to compute ef

Re: [PR] Remove usage of IndexSearcher#search(Query, Collector) from join package [lucene]

2025-01-20 Thread via GitHub
javanna commented on PR #13747: URL: https://github.com/apache/lucene/pull/13747#issuecomment-2601944504 Hey @msfroh I need to review this once again and resume context on it. I will try to get to it this week. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Avoid overflow in index input slices invariant checks [lucene]

2025-01-20 Thread via GitHub
ChrisHegarty merged PR #14126: URL: https://github.com/apache/lucene/pull/14126 -- 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...@lucen