+1 to both adding doAfterFlush and making the two methods protected. Patch?
Mike On Tue, Feb 23, 2010 at 6:55 AM, Shai Erera <ser...@gmail.com> wrote: > Hi, > > Can we add to IW a doBeforeFlush(), similar to doAfterFlush(), which will > get called before flush actually happens (i.e., at the beginning of > flush())? IW.flush() is final and so I cannot override it, but I do take > advantage of doAfterFlush(). I need though a way to execute some code before > the flush begins ... > > On that matter, can we turn doAfterFlush to protected? It's designed to be > an extension point, yet still package private. Since it does not include any > logic (as well as doBeforeFlush shouldn't), and all we'll need to guarantee > is that we'll call both before and after flush, I think it'll make sense to > make these two 'public' in nature, i.e. protected visibility. > > What do you think? > > Shai > --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org For additional commands, e-mail: java-dev-h...@lucene.apache.org
