Hi Davide,

So far the intention was that we would soon merge LucenePropertyIndex
and LuceneIndex into one and not have duplicate logic. However to
avoid the confusion above suggestion make sense and I have done them
as part of OAK-2239. Can you try with those changes and see if it
helps
Chetan Mehrotra


On Tue, Oct 28, 2014 at 9:20 PM, Davide Giannella <[email protected]> wrote:
> Hello team,
>
> while debugging for OAK-2226 (with Thomas' help) I discovered two points:
>
> LucenePropertyIndex.getIndexName() always return "lucene" as for the
> already existing Lucene index implementation. Returning something like
> "lucene-property" or similar would ease to understand the traces in the
> debug statements.
>
> Then the LucenePropertyIndex is wrapped within the aggregator. Does it
> make sense to have an aggregated property index?
>
> Cheers
> Davide
>
>

Reply via email to