[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-20 Thread Erick Erickson (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16056272#comment-16056272 ] Erick Erickson commented on LUCENE-7880: bq: It wouldn't be an issue if it was a

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-20 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16056082#comment-16056082 ] Adrien Grand commented on LUCENE-7880: -- bq. If you were writing this from scratch wo

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-20 Thread Alan Woodward (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055988#comment-16055988 ] Alan Woodward commented on LUCENE-7880: --- bq. we should find a heuristic that can de

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-20 Thread Erick Erickson (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055916#comment-16055916 ] Erick Erickson commented on LUCENE-7880: Adrien: So how would you fix the design

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-20 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055780#comment-16055780 ] Adrien Grand commented on LUCENE-7880: -- Rewriting to a TermInSetQuery would not work

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-20 Thread Alan Woodward (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055443#comment-16055443 ] Alan Woodward commented on LUCENE-7880: --- I've run into the limit when building clas

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-20 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055402#comment-16055402 ] Adrien Grand commented on LUCENE-7880: -- Agreed with you David that it is similar to

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-19 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055126#comment-16055126 ] Mark Miller commented on LUCENE-7880: - The difference is probably that people that al

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-19 Thread David Smiley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16055078#comment-16055078 ] David Smiley commented on LUCENE-7880: -- By the way, this {{maxBooleanClauseCount}} i

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-19 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16054831#comment-16054831 ] Adrien Grand commented on LUCENE-7880: -- I think the source of the disagreement is th

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-19 Thread Yonik Seeley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16054250#comment-16054250 ] Yonik Seeley commented on LUCENE-7880: -- First, lets assume that having this global l

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-18 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16053319#comment-16053319 ] Adrien Grand commented on LUCENE-7880: -- Mark, what is your proposal? This issue loo

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Erick Erickson (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16053017#comment-16053017 ] Erick Erickson commented on LUCENE-7880: bq: This is not the only arbitrary limit

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Mark Miller (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16053018#comment-16053018 ] Mark Miller commented on LUCENE-7880: - I'm all for trappy limits that pop up out of t

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Yonik Seeley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16053001#comment-16053001 ] Yonik Seeley commented on LUCENE-7880: -- bq. This is not the only arbitrary limit we

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Yonik Seeley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16052999#comment-16052999 ] Yonik Seeley commented on LUCENE-7880: -- bq. users can already check the number of cl

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16052998#comment-16052998 ] Adrien Grand commented on LUCENE-7880: -- This is not the only arbitrary limit we have

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Erick Erickson (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16052985#comment-16052985 ] Erick Erickson commented on LUCENE-7880: Adrien: We simply cannot create arbitra

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Erick Erickson (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16052964#comment-16052964 ] Erick Erickson commented on LUCENE-7880: +1 My first reaction was that burdening

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Adrien Grand (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16052941#comment-16052941 ] Adrien Grand commented on LUCENE-7880: -- I think of the max clause count as something

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-17 Thread Yonik Seeley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16052888#comment-16052888 ] Yonik Seeley commented on LUCENE-7880: -- Anyone object to the idea of a getter/setter

[jira] [Commented] (LUCENE-7880) Make boolean query clause limit configurable per-query

2017-06-16 Thread David Smiley (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-7880?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16052625#comment-16052625 ] David Smiley commented on LUCENE-7880: -- +1 fine idea. With this new proposed option