Erik Hatcher wrote:
First let me demonstrate an issue. I'm currently using QueryParser with AND as the default operator for one type of user query. For another type of user query I create an OR'd BooleanQuery programatically.

I want all queries, regardless of how they were created, to be converted back into a parsable expression resulting in the same Query when parsed again (ignoring any analyzer issues at this point). For a programmatically created BooleanQuery for "term1 OR term2", the toString method returns:

        term1 term2

This, of course, when parsed again using AND as the default operator, completely changes the query.

My first proposal is to change BooleanQuery.toString() to always put " OR " between terms that are not required or prohibited. Any objections to this change?

Erik,

You seem to be describing two different things, one a user-entered
style and another a "canonical" style. Could we perhaps keep the
current toString() and add a separate method such as toCanonicalString()
or something akin? It seems that (a) altering something so fundamental
may have unintended consequences across the installed base, and (b)
there may be both added benefits and unintended problems with adding
ORs where they didn't exist that could be shaken out over a longer
period of time.

For myself, I can see both a requirement to be able to maintain the
original query and a canonical version. But mixing them seems like
it might cause problems.

My second proposal, and this relates to a custom query parser I'm working on, is to separate the conversion of a Query to a String completely from the Query class itself. I envision some type of abstract factory mechanism, with the default implementation being the existing (mostly QueryParser compatible) format. Query.toString() would delegate to the formatter factory. The formatting implementation would be pluggable to allow other custom query expression formats to be generated from any Query instance.

I haven't fleshed out the exact design yet - I'm soliciting ideas for this effort. Thoughts?

Not any detailed ones, as I'm rushing out the door in a minute, but I agree with the formatter idea and think it is a better solution than altering the toString() method (if I understand both proposals correctly).

Murray

......................................................................
Murray Altheim                    http://kmi.open.ac.uk/people/murray/
Knowledge Media Institute
The Open University, Milton Keynes, Bucks, MK7 6AA, UK               .

 Life as a modern man is dangerous:

    "Laptops can reach internal operating temperatures of over
     70C.  They are frequently positioned close to the scrotum,
     as well as being capable of producing direct local heat,
     they require the user to sit with his thighs close together
     to balance the machine, trapping the scrotum."
     http://www.guardian.co.uk/online/news/0,12597,1369644,00.html

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to