[Yanel-dev] LuceneIndexerV2

[Michael Wechner]

Hi Balz

Thanks very much for your feedback.

I agree we need to continue improving this. Will get back to it when I will
be back in a couple of days.

Cheers

Michael

On 4/29/11 3:18 PM, Balz Schreier wrote:
> Hi Michael,
>
> I am now implementing property indexing for my project.
> I noticed the following in the current LuceneIndexerV2 class:
>
> - the updateIndex method is not synchronized, this can easily cause a 
> corrupt index at high traffic
>
> - the line where the fulltext is added to the document:
>
>                     luceneDoc.add(new 
> Field(LuceneIndexer.INDEX_PROPERTY_FULL, fullText, Field.Store.YES, 
> Field.Index.TOKENIZED));
>
>
> This has changed compared to LuceneIndexer (v1): there it was stored 
> with Field.Store.NO <http://Field.Store.NO> .
> Is there a certain reason why this has been changed? I only ask 
> because this will increase dramatically the size of the index if you 
> store the full content in the index too.
>
>
>
> As I will use my own Indexer class anyway, this has no urgency to be 
> changed in Yanel, but it is interesting to improve the V2 class.
>
> Cheers
> Balz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wyona.org/pipermail/yanel-development/attachments/20110430/0408a5d9/attachment.html>