Mailing-List: contact dev-help@lucene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@lucene.apache.org
Message-ID: <9180832.549321286294073660.JavaMail.jira@thor>
Date: Tue, 5 Oct 2010 11:54:33 -0400 (EDT)
From: "Shai Erera (JIRA)" <jira@apache.org>
To: dev@lucene.apache.org
Subject: [jira] Commented: (LUCENE-2689) remove NativeFSLockFactory's
 attempt to acquire a test lock
In-Reply-To: <13146677.549051286293354182.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-2689?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12918036#action_12918036 ] 

Shai Erera commented on LUCENE-2689:
------------------------------------

All tests passed for me.

BTW, according to the code, this method was added due to NFS cache issues, but I agree that it's useless to call makeLock just to call obtain() some time later on.  It's almost as if we should make makeLock obtain the lock right away. Is there a reason why we don't do it? Is it essential to separate it into two calls?

> remove NativeFSLockFactory's attempt to acquire a test lock
> -----------------------------------------------------------
>
>                 Key: LUCENE-2689
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2689
>             Project: Lucene - Java
>          Issue Type: Bug
>    Affects Versions: 3.0.1
>            Reporter: Michael McCandless
>            Assignee: Michael McCandless
>             Fix For: 3.1, 4.0
>
>
> NativeFSLockFactory tries to acquire a test lock the first time a lock is created.  It's the only LF to do this, and, it's caused us hassle (LUCENE-2421,  LUCENE-2688).
> I think we should just remove it.  The caller of .makeLock will presumably immediately thereafter acquire the lock and at the point hit any exception that acquireTestLock would've hit.

-- 
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