[ 
https://issues.apache.org/jira/browse/LUCENE-8845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16860689#comment-16860689
 ] 

Alan Woodward commented on LUCENE-8845:
---------------------------------------

I can add the following:
{code}
diff --git 
a/lucene/sandbox/src/java/org/apache/lucene/search/intervals/MultiTermIntervalsSource.java
 
b/lucene/sandbox/src/java/org/apache/lucene/search/intervals/MultiTermIntervalsSource.java
index 213ef730476..9c9b5f95c28 100644
--- 
a/lucene/sandbox/src/java/org/apache/lucene/search/intervals/MultiTermIntervalsSource.java
+++ 
b/lucene/sandbox/src/java/org/apache/lucene/search/intervals/MultiTermIntervalsSource.java
@@ -27,6 +27,7 @@ import java.util.Objects;
 import org.apache.lucene.index.LeafReaderContext;
 import org.apache.lucene.index.Terms;
 import org.apache.lucene.index.TermsEnum;
+import org.apache.lucene.search.BooleanQuery;
 import org.apache.lucene.search.MatchesIterator;
 import org.apache.lucene.search.MatchesUtils;
 import org.apache.lucene.search.QueryVisitor;
@@ -41,6 +42,10 @@ class MultiTermIntervalsSource extends IntervalsSource {

   MultiTermIntervalsSource(CompiledAutomaton automaton, int maxExpansions, 
String pattern) {
     this.automaton = automaton;
+    if (maxExpansions > BooleanQuery.getMaxClauseCount()) {
+      throw new IllegalArgumentException("maxExpansions [" + maxExpansions
+          + "] cannot be greater than BooleanQuery.getMaxClauseCount [" + 
BooleanQuery.getMaxClauseCount() + "]");
+    }
     this.maxExpansions = maxExpansions;
     this.pattern = pattern;
   }
{code}

> Allow maxExpansions to be set on multi-term Intervals
> -----------------------------------------------------
>
>                 Key: LUCENE-8845
>                 URL: https://issues.apache.org/jira/browse/LUCENE-8845
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Alan Woodward
>            Assignee: Alan Woodward
>            Priority: Major
>             Fix For: 8.2
>
>         Attachments: LUCENE-8845.patch
>
>
> MultiTermIntervalsSource has a maxExpansions parameter which is always set to 
> 128 by the factory methods Intervals.prefix() and Intervals.wildcard().  We 
> should keep 128 as the default, but also add additional methods that take a 
> configurable maximum.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to