Mailing-List: contact dev-help@lucene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@lucene.apache.org
Message-ID: <31629570.490451282418475744.JavaMail.jira@thor>
Date: Sat, 21 Aug 2010 15:21:15 -0400 (EDT)
From: "Shai Erera (JIRA)" <jira@apache.org>
To: dev@lucene.apache.org
Subject: [jira] Resolved: (LUCENE-2592) Add static readSnapshotsInfo to
 PersistentSnapshotDeletionPolicy
In-Reply-To: <24536576.251691281436455741.JavaMail.jira@thor>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/LUCENE-2592?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Shai Erera resolved LUCENE-2592.
--------------------------------

    Resolution: Fixed

Committed revision 987811 (3x).
Committed revision 987814 (trunk).

> Add static readSnapshotsInfo to PersistentSnapshotDeletionPolicy
> ----------------------------------------------------------------
>
>                 Key: LUCENE-2592
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2592
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Index
>            Reporter: Shai Erera
>            Assignee: Shai Erera
>            Priority: Minor
>             Fix For: 3.1, 4.0
>
>         Attachments: LUCENE-2592.patch
>
>
> PSDP persists the snapshots information in a Directory. When you open PSDP, it obtains a write lock on the snapshots dir (by keeping an open IndexWriter), and updates the directory when snapshots are created/released.
> This causes problem in the following scenario -- you have two processes, one updates the 'content' index and keeps PSDP open (because it also takes snapshots). Another process wants to read the existing snapshots information and open a read-only IndexReader on the 'content' index. The other process cannot read the existing snapshots information, because p1 keeps a write lock on the snapshots directory.
> There are two possible solutions:
> # Have PSDP open the IndexWriter over the directory for each snapshot/release. A bit expensive, and unnecessary.
> # Introduce a static readSnapshotsInfo on PSDP which accepts a Directory and returns the snapshots information. IMO it's cleaner, and won't have the performance overhead of opening/closing the IW as before.
> I'll post a patch (implementing the 2nd approach) shortly. I'd appreciate any comments.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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