Re: AnalyzingInfixSuggester questions

2014-08-15 Thread Michael Sokolov
I opened LUCENE-5889. While I was writing that I remembered another gap: it would be useful to have a delete() method. Callers might like to be able to include or exclude terms based on their frequency, which can change over time, and at the very least when a term no longer occurs in the sourc

Re: AnalyzingInfixSuggester questions

2014-08-15 Thread Michael Sokolov
On 8/14/2014 5:48 PM, Michael McCandless wrote: I think we should expose commit? Can you open an issue? I will And fix the NPEs to be a more informative exc? E.g. IllegalStateException w/ a reasonable message ... You should be able to init AIS on a previously built suggester, not .build() it

Re: AnalyzingInfixSuggester questions

2014-08-14 Thread Michael McCandless
I think we should expose commit? Can you open an issue? And fix the NPEs to be a more informative exc? E.g. IllegalStateException w/ a reasonable message ... You should be able to init AIS on a previously built suggester, not .build() it every time your application starts... Mike McCandless h