lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adriano Crestani (JIRA)" <>
Subject [jira] Commented: (LUCENE-1938) Precedence query parser using the contrib/queryparser framework
Date Thu, 08 Oct 2009 05:07:31 GMT


Adriano Crestani commented on LUCENE-1938:

I looked at the pipeline PrecedenceQueryNodeProcessorPipeline, it contains some complex logic
off removing a processor from the StandardQueryNodeProcessorPipeline.

I think it would be better to have a common ancestor, CommonQueryNodeProcessorPipeline, that
both StandardQueryNodeProcessorPipeline and PrecedenceQueryNodeProcessorPipeline can extend.

I don't think it's so complex, it just removes GroupQueryNodeProcessor and adds ModifierQueryNodeProcessor
(I think this is the class name). I also added this description to the javadocs.

Anyway, I would prefer to leave it like that, extending StandardQPP, instead of creating a
common pipeline. The common pipeline would not make much sense right now, there is not QueryBuilder
impl that would produce a correct Query instance from a query tree processed by it, so no
one could just use it.

In future, we could have a third QP implementation, that instead of replacing GroupQueryNodeProcessor
by ModifierQueryNodeProcessor, it would replace another processor, or switch the processors
position, etc... the CommonQueryNodeProcessorPipeline would not make sense anymore, it only
make sense right now for PrecedenceQP and StandardQP, because we know what both have in common,
but doesn't mean a third QP would have the same in common.

So, +1 to leave it the way it's.

> Precedence query parser using the contrib/queryparser framework
> ---------------------------------------------------------------
>                 Key: LUCENE-1938
>                 URL:
>             Project: Lucene - Java
>          Issue Type: New Feature
>          Components: contrib/*
>    Affects Versions: 2.9
>            Reporter: Adriano Crestani
>            Assignee: Adriano Crestani
>            Priority: Minor
>             Fix For: 3.1
>         Attachments: LUCENE-1938.patch
> Extend the current StandardQueryParser on contrib so it supports boolean precedence

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