Home | About | Sematext search-lucene.com search-hadoop.com
 Search Lucene and all its subprojects:

Switch to Threaded View
Lucene, mail # dev - Re: Small issue in queryparser // QueryNodeImpl.java


Copy link to this message
-
Re: Small issue in queryparser // QueryNodeImpl.java
Adriano Crestani 2011-04-25, 00:07
Hi Karsten,

I received an email when you commented on LUCENE-2867, but I couldn't find
the comment on the JIRA itself.

Anyway, thanks for reporting this bug, I create a JIRA for that
(LUCENE-2045) and already started working on it.

Best Regards,
Adriano Crestani

On Sun, Apr 24, 2011 at 5:49 PM, Karsten Fissmer <[EMAIL PROTECTED]>wrote:

> Hi Adriano Crestani,
>
> I made a comment in
> https://issues.apache.org/jira/browse/LUCENE-2867 :
>
> Since your changes in Rev. 1063065 "applying patches from LUCENE-2855 and
> LUCENE-2867 to trunk" there is a problem with method "containsTag" in class
>
> lucene/contrib/queryparser/src/java/org/apache/lucene/queryParser/core/nodes/QueryNodeImpl.java
>
> old source-code:
>
>  public boolean containsTag(CharSequence tagName) {
>    return this.tags.containsKey(tagName.toString().toLowerCase());
>  }
>
>  public Object getTag(CharSequence tagName) {
>    return this.tags.get(tagName.toString().toLowerCase());
>  }
>
> current source-code:
>
>  /** verify if a node contains a tag */
>  public boolean containsTag(String tagName) {
>    return this.tags.containsKey(tagName);
>  }
>
>  public Object getTag(String tagName) {
>    return this.tags.get(tagName.toString().toLowerCase());
>  }
>
> so there is a missing ".toLowerCase()".
>
>
>
> Best regards
>
>  Karsten
>  http://www.xing.com/profile/Karsten_Fissmer
>
>
> P.S. currently I am working on new QueryBuilder classes to generate
> SpanQueries from QueryNodes. Is this work already down by someone else?
>