cxf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Beryozkin (JIRA)" <>
Subject [jira] [Commented] (CXF-5938) LuceneQueryVisitior is not reusable / not thread-safe
Date Tue, 12 Aug 2014 14:07:11 GMT


Sergey Beryozkin commented on CXF-5938:

Hi Andriy, 
It occurred to me a new SearchConditionVisitor.reset() method would be redundant, Lucene (and
JPA) visitors should simply reset the state whenever their visit(SearchCondition) is called,
do you agree ? reset() method would affect the custom SearchConditionVisitor implementations
too, so IMHo we'd better avoid adding it.
Thanks, Sergey

> LuceneQueryVisitior is not reusable / not thread-safe
> -----------------------------------------------------
>                 Key: CXF-5938
>                 URL:
>             Project: CXF
>          Issue Type: Bug
>          Components: Integration
>    Affects Versions: 3.0.1
>            Reporter: Andriy Redko
>            Assignee: Andriy Redko
>            Priority: Minor
>             Fix For: 3.0.2
> LuceneQueryVisitor class is not really reusable in current implementation: it keeps the
state of all parsed queries (which is generally fine) but it groups them by property name,
returning the first query from the list all the time. That means running two search criteria
like 'ct=java' and 'ct=websockets' causes the result of 'ct=java' to be returned in both cases
(very easy reproducible).

This message was sent by Atlassian JIRA

View raw message