[GitHub] [lucene] twosom closed pull request #12044: some cleanup and refactoring codes of analysis-nori & analysis-kuromoji

2023-02-18 Thread via GitHub
twosom closed pull request #12044: some cleanup and refactoring codes of analysis-nori & analysis-kuromoji URL: https://github.com/apache/lucene/pull/12044 -- 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

[GitHub] [lucene] twosom closed pull request #12040: Minor refactoring and cleanup to BooleanQuery code

2023-02-18 Thread via GitHub
twosom closed pull request #12040: Minor refactoring and cleanup to BooleanQuery code URL: https://github.com/apache/lucene/pull/12040 -- 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 co

[GitHub] [lucene] gsmiller opened a new pull request, #12155: Bring "max ord" check optimization in SortedSetDocValuesSetQuery over…

2023-02-18 Thread via GitHub
gsmiller opened a new pull request, #12155: URL: https://github.com/apache/lucene/pull/12155 … to DocValuesRewriteMethod ### Description This brings over the recent "max ord" optimization in `SortedSetDocValuesSetQuery` (#12129) to `DocValuesRewriteMethod`. -- This is an au

[GitHub] [lucene] gsmiller opened a new pull request, #12156: Remove custom TermInSetQuery implementation in favor of extending MultiTermQuery

2023-02-18 Thread via GitHub
gsmiller opened a new pull request, #12156: URL: https://github.com/apache/lucene/pull/12156 ## Description `TermInSetQuery`'s implementation is more-or-less an exactly copy of `MultiTermQuery` + `MultiTermQueryConstantScoreWrapper`. This PR removes the custom `TermInSetQuery` imple

[GitHub] [lucene] gsmiller commented on pull request #12055: Better skipping for multi-term queries with a FILTER rewrite.

2023-02-18 Thread via GitHub
gsmiller commented on PR #12055: URL: https://github.com/apache/lucene/pull/12055#issuecomment-1435802423 As one more update, I just opened a new PR to have `TermInSetQuery` extend `MultiTermQuery` instead of having its own custom implementation (#12156). I was able to do some benchmarking