Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 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
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?
>
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB