Re: Question regarding escaped sequence

Erik Hatcher Wed, 03 Mar 2004 05:59:18 -0800

Otis,

Just a public "thanks!" for applying these patches. If there were more hours in a day I'd have been more proactive with it myself.

Erik

On Mar 3, 2004, at 7:33 AM, Otis Gospodnetic wrote:

I closed the latter two, but the first one is a JMeter bug.
Thanks for your work, I think this fix will make several people happy!

Otis

--- Jean-Francois Halleux <[EMAIL PROTECTED]> wrote:

Otis, you can probably close bugs 16370, 11636, and 14665 as well.

Have a look at those too.

KR,

Jeff

-----Original Message-----
From: Otis Gospodnetic [mailto:[EMAIL PROTECTED]
Sent: mercredi 3 mars 2004 12:19
To: Lucene Developers List
Subject: Re: Question regarding escaped sequence


This indeed fixes the bug that the code further below demonstrates,
so
I'm comitting it.

http://issues.apache.org/bugzilla/show_bug.cgi?id=24665

Otis

--- Otis Gospodnetic <[EMAIL PROTECTED]> wrote:

I have a feeling that query escaping really is broken in Lucene.
Try running the class below like this:

prompt> java Escaper '+string' '\+string'

I get:

$ java Escaper '+string' '\+string'
0: +string
1: \+string
QUERY: \+string
HITS: 0

That should give me 1 hit, shouldn't it?

import org.apache.lucene.queryParser.QueryParser;
import org.apache.lucene.analysis.*;
import org.apache.lucene.search.*;
import org.apache.lucene.index.*;
import org.apache.lucene.store.*;
import org.apache.lucene.document.*;

public class Escaper
{
    public static void main(String[] args) throws Exception
    {
        System.out.println("0: " + args[0]);
        System.out.println("1: " + args[1]);

        Directory dir = new RAMDirectory();
        IndexWriter writer = new IndexWriter(dir, new
WhitespaceAnalyzer(), true);
        Document doc = new Document();
        doc.add(Field.Text("text", args[0]));
        writer.addDocument(doc);
        writer.optimize();
        writer.close();

        QueryParser qp = new QueryParser("text", new
WhitespaceAnalyzer());
        Query q = qp.parse(args[1]);
        System.out.println("QUERY: " + q.toString("text"));

        IndexSearcher searcher = new IndexSearcher(dir);
        Hits hits = searcher.search(q);
        System.out.println("HITS: " + hits.length());
        searcher.close();
    }
}

Thanks,
Otis

--- Jean-Francois Halleux <[EMAIL PROTECTED]> wrote:

Hello,

in TestQueryParser, method testEscaped(), I see the following:

...
assertQueryEquals("\\+blah", a, "\\+blah");
assertQueryEquals("\\(blah", a, "\\(blah");

assertQueryEquals("\\-blah", a, "\\-blah");
assertQueryEquals("\\!blah", a, "\\!blah");
assertQueryEquals("\\{blah", a, "\\{blah");
assertQueryEquals("\\}blah", a, "\\}blah");
...

is this really the expected behavior? Shouldn't \\-blah be
interpreted
as -blah and \\!blah as !blah ?

Thanks,

Jean-Francois Halleux

---------------------------------------------------------------------

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

[EMAIL PROTECTED]

---------------------------------------------------------------------

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

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

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

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

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

Re: Question regarding escaped sequence

Reply via email to