Mailing-List: contact dev-help@lucene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@lucene.apache.org
Received-SPF: neutral (athena.apache.org: local policy)
MIME-Version: 1.0
In-Reply-To: <t2m786fde51004270659r63abf85en359b07ae00fcd75d@mail.gmail.com>
References: <t2m786fde51004270659r63abf85en359b07ae00fcd75d@mail.gmail.com>
Date: Tue, 27 Apr 2010 10:54:58 -0400
Message-ID: <g2v9ac0c6aa1004270754x2c664e74z190e6ffbe4ceaf6b@mail.gmail.com>
Subject: Re: Enhance SnapshotDeletionPolicy to allow taking multiple snapshots
From: Michael McCandless <lucene@mikemccandless.com>
To: dev@lucene.apache.org
Content-Type: text/plain; charset=ISO-8859-1

This would be great!

I think we should just enhance SDP rather than make a new MDP.

Mike

On Tue, Apr 27, 2010 at 9:59 AM, Shai Erera <serera@gmail.com> wrote:
> Hi
>
> Today, SnapshotDeletionPolicy (SDP) allows taking only one snapshot. I need
> to be able to take multiple snapshots. Consider multiple processes doing
> several things on the index - each needs a snapshot of the index so that
> commit it needs doesn't get deleted under the covers. SDP is perfect, only
> it allows for only one snapshot. So I was thinking to extend it to a
> MultiSDP which adds an 'id' parameter to snapshot() and release(). But then,
> I was thinking - why shouldn't that exist in SDP? It won't make the API
> anymore complicated, and in addition won't introduce yet another DP class.
>
> This can be done in two ways:
> 1) snapshot() and release() get the extra parameter. For convenience we can
> allow for a null id, in which case only one snapshot w/ a null id can be
> taken (until it's released). To avoid making up an id if you really need a
> single snapshot.
> 2) add variants snapshot() and release() which take an id as argument. Or
> ... extend SDP to MultiSDP.
>
> I'd prefer if we keep that functionality in SDP, but if you prefer an
> extension to it, then we'll need to allow for easier extension of SDP (I
> think we should do that anyway).
>
> What do you think?
>
> Shai
>

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