Re: [PR] Made DocIdsWriter use DISI when reading documents with an IntersectVisitor [lucene]

2024-03-12 Thread via GitHub
antonha commented on PR #13149: URL: https://github.com/apache/lucene/pull/13149#issuecomment-1993673162 I spent some time "proving" this is luceneutil - [luceneutil/pull/257](https://github.com/mikemccand/luceneutil/pull/257) adds a reproduction - if run with `wikimediumall`, `optimize = T

Re: [PR] Add new parallel merge task executor for parallel actions within a single merge action [lucene]

2024-03-12 Thread via GitHub
zhaih commented on code in PR #13124: URL: https://github.com/apache/lucene/pull/13124#discussion_r1522601711 ## lucene/core/src/java/org/apache/lucene/index/MergeScheduler.java: ## @@ -52,6 +56,14 @@ public Directory wrapForMerge(OneMerge merge, Directory in) { return in;

Re: [PR] Move synonym map off-heap for SynonymGraphFilter [lucene]

2024-03-12 Thread via GitHub
github-actions[bot] commented on PR #13054: URL: https://github.com/apache/lucene/pull/13054#issuecomment-1992798063 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 MultiTermQueryDocValuesWrapper not propagating QueryVisitor [lucene]

2024-03-12 Thread via GitHub
github-actions[bot] commented on PR #13140: URL: https://github.com/apache/lucene/pull/13140#issuecomment-1992797629 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] Add new parallel merge task executor for parallel actions within a single merge action [lucene]

2024-03-12 Thread via GitHub
benwtrent commented on PR #13124: URL: https://github.com/apache/lucene/pull/13124#issuecomment-1992172143 @jpountz OK, so I did some benchmarking to look at the impact of this change. 500k docs, flushing segments every 1MB and then force merging. For all of these, the number

Re: [PR] Support disabling IndexSearcher.maxClauseCount with a value of -1 [lucene]

2024-03-12 Thread via GitHub
dsmiley commented on code in PR #13178: URL: https://github.com/apache/lucene/pull/13178#discussion_r1521507598 ## lucene/core/src/java/org/apache/lucene/search/QueryVisitor.java: ## @@ -99,5 +99,11 @@ public void consumeTerms(Query query, Term... terms) { } /** A QueryV

Re: [I] TestIDVersionPostingsFormat failure [lucene]

2024-03-12 Thread via GitHub
benwtrent closed issue #13127: TestIDVersionPostingsFormat failure URL: https://github.com/apache/lucene/issues/13127 -- 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 unsubsc

Re: [PR] Fix potential race condition in DocumentsWriter & DocumentsWriterDeleteQueue [lucene]

2024-03-12 Thread via GitHub
benwtrent merged PR #13169: URL: https://github.com/apache/lucene/pull/13169 -- 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.a

Re: [PR] Add new parallel merge task executor for parallel actions within a single merge action [lucene]

2024-03-12 Thread via GitHub
jpountz commented on code in PR #13124: URL: https://github.com/apache/lucene/pull/13124#discussion_r1521095208 ## lucene/core/src/java/org/apache/lucene/index/ConcurrentMergeScheduler.java: ## @@ -910,4 +941,58 @@ public void setSuppressExceptions(ConcurrentMergeScheduler cms)