lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Otis Gospodnetic (JIRA)" <>
Subject [jira] Commented: (LUCENE-883) make spell checker test case work again
Date Fri, 18 May 2007 04:37:16 GMT


Otis Gospodnetic commented on LUCENE-883:

Do you need reader=null?
Do you need to close reader explicitly here?  searcher.close() can close it, depending on
which searcher ctor was used, if I recall correctly.  Not sure which one is used in spellchecker,
off the top of my head.

Re Solr - yes, I added that to Solr in SOLR-81.

> make spell checker test case work again
> ---------------------------------------
>                 Key: LUCENE-883
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Bug
>    Affects Versions: 2.1
>            Reporter: Daniel Naber
>         Attachments: lucene-spellchecker-2.diff
> See attached path which makes the spellchecker test case work again. The problem without
the patch is that consecutive calls to indexDictionary() will create a spelling index with
duplicate words. Does anybody see a problem with this patch? I see that the spellchecker code
is now used in Solr, isn't it? I didn't have time to test this patch inside Solr.
> Also see, but the null check is included
in this patch so the NPE described there cannot happen anymore.

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