[LUCENE-2108] SpellChecker file descriptor leak - no way to close the IndexSearcher used by SpellChecker internally

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0
• Fix Version/s: 3.0.1, 4.0-ALPHA
• Component/s: modules/spellchecker
• Lucene Fields: New, Patch Available
• Assignee: Simon Willnauer
• Reporter: Eirik Bjorsnos
• Created: 03/Dec/09 20:07
• Updated: 28/Aug/22 12:15
• Resolved: 05/Dec/09 12:19

Description

I can't find any way to close the IndexSearcher (and IndexReader) that
is being used by SpellChecker internally.

I've worked around this issue by keeping a single SpellChecker open
for each index, but I'd really like to be able to close it and
reopen it on demand without leaking file descriptors.

Could we add a close() method to SpellChecker that will close the
IndexSearcher and null the reference to it? And perhaps add some code
that reopens the searcher if the reference to it is null? Or would
that break thread safety of SpellChecker?

The attached patch adds a close method but leaves it to the user to
call setSpellIndex to reopen the searcher if desired.

Issue Links

• blocks: SOLR-785 Distributed SpellCheckComponent