[GitHub] [lucene] benwtrent commented on pull request #12197: [Backport] GITHUB-11838 Add api to allow concurrent query rewrite

2023-03-31 Thread via GitHub
benwtrent commented on PR #12197: URL: https://github.com/apache/lucene/pull/12197#issuecomment-1491809549 Heya @uschindler didn't want this to die on the vine. Hope your trip was excellent! -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [lucene] benwtrent opened a new issue, #12219: Making vector similarity functions pluggable

2023-03-31 Thread via GitHub
benwtrent opened a new issue, #12219: URL: https://github.com/apache/lucene/issues/12219 ### Description There are two major reasons for adding a custom vector similarity function: - Adding new nuanced similarity functions (jaccard, hamming, etc.) - Have external support fo

[GitHub] [lucene] jpountz commented on a diff in pull request #12216: Adjust DWPT pool concurrency to the number of cores.

2023-03-31 Thread via GitHub
jpountz commented on code in PR #12216: URL: https://github.com/apache/lucene/pull/12216#discussion_r1154392550 ## lucene/core/src/java/org/apache/lucene/index/ConcurrentApproximatePriorityQueue.java: ## @@ -22,25 +22,40 @@ /** * Concurrent version of {@link ApproximatePrio

[GitHub] [lucene] jpountz merged pull request #12216: Adjust DWPT pool concurrency to the number of cores.

2023-03-31 Thread via GitHub
jpountz merged PR #12216: URL: https://github.com/apache/lucene/pull/12216 -- 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.apa

[GitHub] [lucene] jpountz commented on issue #12219: Making vector similarity functions pluggable

2023-03-31 Thread via GitHub
jpountz commented on issue #12219: URL: https://github.com/apache/lucene/issues/12219#issuecomment-1492296110 What about doing this through vector formats: vector formats could take a similarity parameter, which would win over the one configured on the field type when set. The benefit is th

[GitHub] [lucene] MarcusSorealheis commented on issue #12178: TermAutomatonQuery explain() should return relevant explain output instead of null

2023-03-31 Thread via GitHub
MarcusSorealheis commented on issue #12178: URL: https://github.com/apache/lucene/issues/12178#issuecomment-1492574278 > @MarcusSorealheis looks reasonable to me, thanks for taking it on! Anytime! Happy to help and more to come. 😁 -- This is an automated message from the Apache Git