lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mark Miller (JIRA)" <>
Subject [jira] Commented: (LUCENE-1819) MatchAllDocsQuery.toString(String field) does not honor the javadoc contract
Date Tue, 18 Aug 2009 15:41:14 GMT


Mark Miller commented on LUCENE-1819:

bq. What are you thoughts about QueryParser being able to know about custom Query implementations?
E.g. if I were to write a MyQuery class and implemen the toString method a certain way, how
would QueryParser know about MyQuery?

Right - this has always been a limitation. That is why the wording has been loosened a lot
of the years in Query.toString(). There is really not a lot you can do here - non of the SpanQuerys
produce parsable toStrings because there is no support for SpanQuerys in the QueryParser.
I think a couple others do not work either. I think its just got to be something we live with
- you simply can't count on it - but if possible, its nice I guess - especially for the core

bq.  Is it possible to extend QueryParser?

Its possible to extend it, but not really syntax wise. The new QueryParser in contrib will
likely be more extensable in that regard in a way - though still you would be building another
parser - it would just hopefully allow for a lot of non cut and paste reuse if you just want
to add syntax (I think - I'm only at the surface with the new QP).

Overall though, its not really a contract, because its not something users can adhere to.
Its more of an internal nicety suggestion - but if your not a Query already in the QP, your
out of luck anyway - at best you could toString to a primitive query parsable String if your
custom query rewrites to primitives.

> MatchAllDocsQuery.toString(String field) does not honor the javadoc contract
> ----------------------------------------------------------------------------
>                 Key: LUCENE-1819
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Search
>    Affects Versions: 2.4.1
>            Reporter: John Wang
>            Assignee: Mark Miller
>            Priority: Minor
>             Fix For: 2.9
>         Attachments: LUCENE-1819.patch
> Should be 
> public String toString(String field){
>   return "*:*";
> }
> QueryParser needs to be able to parse the String form of this query.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message