Suggesters: payloads and filter predicates

2014-01-08 Thread Oliver Christ
Hi, It's great to see support for payloads in the suggesters - this is really helpful, and pretty much addresses LUCENE-4516. Are there any plans to also support them for WFSTs? We have some cases where we don't need the Analyzer's capabilities (we look up the completion using the payload infor

Re: How is incrementToken supposed to detect the lack of reset()?

2014-01-08 Thread Benson Margulies
I'm not in the delegate business, just a straight subclass. So I think they are complementary. Gimme your github identity, and you are, as far as I am concerned, more than welcome to add a section on delegates. On Wed, Jan 8, 2014 at 7:38 AM, Mindaugas Žakšauskas wrote: > Hi, > > Sure, why not

Re: How is incrementToken supposed to detect the lack of reset()?

2014-01-08 Thread Mindaugas Žakšauskas
Hi, Sure, why not - I'm just not sure if my approach (of setting reader in reset()) is preferred over yours (using this.input instead of input in ctor)? Or are they both equally good? m. On Wed, Jan 8, 2014 at 12:18 PM, Benson Margulies wrote: > If you'd like to join in on the doc, see > https:

Re: How is incrementToken supposed to detect the lack of reset()?

2014-01-08 Thread Benson Margulies
If you'd like to join in on the doc, see https://github.com/apache/lucene-solr/pull/14/files. I'd be happy to grant you access to push to my fork. On Wed, Jan 8, 2014 at 5:37 AM, Mindaugas Žakšauskas wrote: > Just for the interest, I had a similar problem too as well as other > people [1]. In my

Re: How is incrementToken supposed to detect the lack of reset()?

2014-01-08 Thread Mindaugas Žakšauskas
Just for the interest, I had a similar problem too as well as other people [1]. In my project, I am extending the Tokenizer class and have another tokenizer (e.g. ClassicTokenizer) as a delegate. Unfortunately, properly overriding all public/protected methods is *not* enough, e.g.: public void res

Re: Slow Index Writes

2014-01-08 Thread Klaus Schaefers
THX! On Wed, Jan 8, 2014 at 10:10 AM, Michael McCandless < luc...@mikemccandless.com> wrote: > NRTManager was renamed to ControlledRealTimeReopenThread at some point. > > But likely simple NRT readers (as Ian described, using > .openIfChanged()) will fit your usage. > > ControlledRealTimeReopenT

Re: Slow Index Writes

2014-01-08 Thread Michael McCandless
NRTManager was renamed to ControlledRealTimeReopenThread at some point. But likely simple NRT readers (as Ian described, using .openIfChanged()) will fit your usage. ControlledRealTimeReopenThread is only necessary if you require certain searches to be real-time, e.g. you just indexed a document