You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Benson Margulies (Created) (JIRA)" <ji...@apache.org> on 2012/04/20 19:26:40 UTC

[jira] [Created] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Add syntax for DisjunctionMaxQuery to the Query Parser
------------------------------------------------------

                 Key: LUCENE-4004
                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
             Project: Lucene - Java
          Issue Type: Improvement
          Components: core/queryparser
    Affects Versions: 3.6
            Reporter: Benson Margulies


I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259227#comment-13259227 ] 

Benson Margulies commented on LUCENE-4004:
------------------------------------------

I note that the xml parser doesn't do DisjunctionMaxQuery. I can't see a reason why this would be controversial, so I'll start with a patch for that.
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Robert Muir (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259273#comment-13259273 ] 

Robert Muir commented on LUCENE-4004:
-------------------------------------

Committed revision 1328981 for the xml queryparser support. Thanks again. 
We can either keep this issue open for the other QPs, or spin off new issues, 
whichever you prefer.

                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>            Assignee: Robert Muir
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259246#comment-13259246 ] 

Benson Margulies commented on LUCENE-4004:
------------------------------------------

Using the syntax ( q | q | q ) might be doable in javacc, but I worry that it's undesirable.

Right now, what's in parens are boolean clauses (with +/-). The insides of a disjunct aren't boolean clauses, they are queries. This could be pretty confusing all around. It would be really better to introduce some syntax that allows for various sorts of grouping, but I don't want to step on the Solr parser's use of {}. Further, DisjunctionMaxQuery is just one thing, and using up | for it seems ill-advised.

% isn't doing anything for a living, so that an option would be 

(%disjunctionmax q q q q q ) would serve, and also open a door to supporting other things. 

?

                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259254#comment-13259254 ] 

Benson Margulies commented on LUCENE-4004:
------------------------------------------

At the risk of drifting off topic ...

>From where I sit, 4.0 looks a long way from a release. Lots of things with no javadoc, with the experimental tag, etc. (I've mostly been reading in Solr.) Am I underestimating the speed at which the entire lodge of beavers pulls a major release together?
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259302#comment-13259302 ] 

Benson Margulies commented on LUCENE-4004:
------------------------------------------

Please see https://issues.apache.org/jira/browse/LUCENE-4012 for an alternative to adding syntax to any of the existing parsers. I think it makes more sense, myself, but if others see value in continuing the line in here I'm game.

                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>            Assignee: Robert Muir
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Robert Muir (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259261#comment-13259261 ] 

Robert Muir commented on LUCENE-4004:
-------------------------------------

What can I say: documentation is always a weakness.

>From the Solr side, I think most of the documentation tends to be higher level on the wiki, and javadocs
don't get as much attention since its a smaller population hacking on the APIs.

>From the Lucene side, things tend to be under-documented or out of date.

In any case, just throw up patches and lets get em in. Nobody gets excited to work on docs too much around here unfortunately, probably because there is no momentum and the existing docs are all out of date and not very good. 

On the other hand, just in the past weekend we've made some serious progress on the lucene documentation already: nuked the old out-of-date-forrest stuff, fixed a ton of broken links, added a broken-link checker, linked the javadocs of various modules to each other, brought all the existing docs (minus fileformats) up to speed with 4.0, generating htmlized version of the .txt documents with markdown, etc.

I know there is a lot of underdocumented stuff in lucene, but currently from my own perspective I am working to correct the broken stuff :)

For some reason, its definitely harder to fix the old documentation up to make sense than i would have ever thought, I spent most of the day just bringing http://lucene.apache.org/core/3_6_0/scoring.html up to speed and integrating it into the o.a.l.search package javadocs.

For the stuff you see thats experimental with no javadoc tag... this is really just as bad of a problem, just open issues if you want to help out. We are pretty overwhelmed with things to fix on the documentation side so any help would be appreciated.

                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>            Assignee: Robert Muir
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259249#comment-13259249 ] 

Benson Margulies commented on LUCENE-4004:
------------------------------------------

Note that the patch is to trunk, I'm sort of assuming that you all do 'patch the trunk and then backport as appropriate'.
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Robert Muir (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259251#comment-13259251 ] 

Robert Muir commented on LUCENE-4004:
-------------------------------------

Usually yes, however at this stage we just released 3.6 (intended to be the last minor release in the 3.x series).

So currently we have not yet cut a branch_4x for stable 4.0 and are only working on trunk.

(separately, we also have http://svn.apache.org/repos/asf/lucene/dev/branches/lucene_solr_3_6/ open for bugfixes, to backport any nasty bugs for 3.6.1, 3.6.2, etc: basically 3.x is intended to be in maintenance mode like 2.9.x was)
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259248#comment-13259248 ] 

Benson Margulies commented on LUCENE-4004:
------------------------------------------

FWIW, that's ||, not |, but I still don't want to burn | on disjunction.
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Updated] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benson Margulies updated LUCENE-4004:
-------------------------------------

    Attachment: LUCENE-4004.patch

Support D-m-q in the XML parser.
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Updated] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benson Margulies updated LUCENE-4004:
-------------------------------------

    Attachment: LUCENE-4004.patch

add the boost param I forgot the first time.
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259240#comment-13259240 ] 

Benson Margulies commented on LUCENE-4004:
------------------------------------------

Indeed I don't quite know what sort of syntax would fly for the tieBreaker, I'm off to study the jj file for the classic parser and see if anything analogous presents itself.

Meanwhile, do you want to do XML in a different JIRA from the hard ones, or should I just stack up the patches here?

                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Robert Muir (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259268#comment-13259268 ] 

Robert Muir commented on LUCENE-4004:
-------------------------------------

By the way this patch looks good. Thanks Benson! I plan to commit this in a bit
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>            Assignee: Robert Muir
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Assigned] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Robert Muir (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robert Muir reassigned LUCENE-4004:
-----------------------------------

    Assignee: Robert Muir
    
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>            Assignee: Robert Muir
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Robert Muir (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259247#comment-13259247 ] 

Robert Muir commented on LUCENE-4004:
-------------------------------------

{quote}
Further, DisjunctionMaxQuery is just one thing, and using up | for it seems ill-advised.
{quote}

I think we allow this for OR as well anyway, so it would be ambiguous...?
                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Issue Comment Edited] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Benson Margulies (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259302#comment-13259302 ] 

Benson Margulies edited comment on LUCENE-4004 at 4/22/12 11:41 PM:
--------------------------------------------------------------------

Please see https://issues.apache.org/jira/browse/LUCENE-4012 for an alternative to adding syntax to any of the existing end-user-facing parsers. I think it makes more sense, myself, but if others see value in continuing the line in here I'm game.

                
      was (Author: bmargulies):
    Please see https://issues.apache.org/jira/browse/LUCENE-4012 for an alternative to adding syntax to any of the existing parsers. I think it makes more sense, myself, but if others see value in continuing the line in here I'm game.

                  
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>            Assignee: Robert Muir
>         Attachments: LUCENE-4004.patch, LUCENE-4004.patch
>
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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


[jira] [Commented] (LUCENE-4004) Add syntax for DisjunctionMaxQuery to the Query Parser

Posted by "Robert Muir (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/LUCENE-4004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259234#comment-13259234 ] 

Robert Muir commented on LUCENE-4004:
-------------------------------------

I'm +1 to adding syntax to this to the classic/flexible/qps as well.

But I like your idea of adding it to the xml one first. Maybe its an easier iteration to then boil down the best
syntax for the other ones (which is going to be more difficult just because they are hairier).

                
> Add syntax for DisjunctionMaxQuery to the Query Parser
> ------------------------------------------------------
>
>                 Key: LUCENE-4004
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4004
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: core/queryparser
>    Affects Versions: 3.6
>            Reporter: Benson Margulies
>
> I've come up with a use for DisjunctionMaxQuery, but not the dismax parser. I note that the toString() method on that item proposes a syntax with vertical bars. Is there any sympathy for a patch that added this to the standard parser, or some other syntax?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

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