lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael McCandless <luc...@mikemccandless.com>
Subject Re: MergePolicy Thresholds
Date Tue, 03 May 2011 09:36:48 GMT
Looks good Shai!

Comments below too:

On Tue, May 3, 2011 at 5:29 AM, Shai Erera <serera@gmail.com> wrote:
> Hi
>
> I looked into porting it to 3x, and prepared the attached patch. It only
> contains the new TieredMP and Test, as well as the necessary changes to
> LuceneTestCase and IndexWriter. I guess you can start with it (even just the
> MP and IW changes) to test it on your indexes.
>
> Mike, I saw that there were many more changes, as part of LUCENE-1076, done
> to the code. In particular, this MP is now the default (on trunk), so I
> guess many changes (to tests) were needed because of that. Do you remember,
> if apart from the changes I've included in the patch, other important
> changes w.r.t. this code?

The only other changes I can think of were some verbosity improvements
to IndexWriter, to support the python script that can make a merge
movie from an infoStream output; but that can wait for when I
back-port to 3.x...

> As we won't change the default MP on 3x, I'm guessing I don't need to port
> all the changes to 3x.

Right, I think.

Mike

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message