Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-27 Thread via GitHub
github-actions[bot] commented on PR #14349: URL: https://github.com/apache/lucene/pull/14349#issuecomment-2759877032 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] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-13 Thread via GitHub
jpountz commented on PR #14349: URL: https://github.com/apache/lucene/pull/14349#issuecomment-2721204272 Implementing it as a query that rewrites to the proper query based on the list of terms makes sense to me. You may want to move this query to the lucene-queries module since it doesn't r

Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
rmuir commented on PR #14349: URL: https://github.com/apache/lucene/pull/14349#issuecomment-2719519004 In my head, that's what we need. There is a crazy difference in construction and execution time between a "native" union and using the efficient linear-time algorithm, as opposed to going

Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
msfroh commented on PR #14349: URL: https://github.com/apache/lucene/pull/14349#issuecomment-2719491949 @willdickerson -- I took a stab at modifying `StringsToAutomaton`, to support case-insensitive matching: https://github.com/apache/lucene/pull/14350 -- This is an automated message from

Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
msfroh commented on code in PR #14349: URL: https://github.com/apache/lucene/pull/14349#discussion_r1992372705 ## lucene/core/src/test/org/apache/lucene/search/TestCaseInsensitiveTermInSetQuery.java: ## @@ -0,0 +1,377 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
msfroh commented on code in PR #14349: URL: https://github.com/apache/lucene/pull/14349#discussion_r1992424195 ## lucene/core/src/java/org/apache/lucene/search/CaseInsensitiveTermInSetQuery.java: ## @@ -81,58 +89,95 @@ public void visit(QueryVisitor visitor) { visitor.con

Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
willdickerson commented on code in PR #14349: URL: https://github.com/apache/lucene/pull/14349#discussion_r1992405437 ## lucene/core/src/test/org/apache/lucene/search/TestCaseInsensitiveTermInSetQuery.java: ## @@ -0,0 +1,377 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
rmuir commented on code in PR #14349: URL: https://github.com/apache/lucene/pull/14349#discussion_r1992333636 ## lucene/core/src/java/org/apache/lucene/search/CaseInsensitiveTermInSetQuery.java: ## @@ -0,0 +1,185 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
rmuir commented on code in PR #14349: URL: https://github.com/apache/lucene/pull/14349#discussion_r1992332504 ## lucene/core/src/java/org/apache/lucene/search/CaseInsensitiveTermInSetQuery.java: ## @@ -0,0 +1,185 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[PR] Case-insensitive TermInSetQuery Implementation (Proof of Concept) [lucene]

2025-03-12 Thread via GitHub
willdickerson opened a new pull request, #14349: URL: https://github.com/apache/lucene/pull/14349 ## Overview This PR introduces a proof of concept for a case-insensitive variant of TermInSetQuery. The implementation provides an efficient way to search for terms regardless of case with