Return-Path: Delivered-To: apmail-lucene-java-dev-archive@www.apache.org Received: (qmail 34792 invoked from network); 3 Feb 2007 18:26:45 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 3 Feb 2007 18:26:45 -0000 Received: (qmail 81551 invoked by uid 500); 3 Feb 2007 18:26:49 -0000 Delivered-To: apmail-lucene-java-dev-archive@lucene.apache.org Received: (qmail 81512 invoked by uid 500); 3 Feb 2007 18:26:49 -0000 Mailing-List: contact java-dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: java-dev@lucene.apache.org Delivered-To: mailing list java-dev@lucene.apache.org Received: (qmail 81501 invoked by uid 99); 3 Feb 2007 18:26:49 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 03 Feb 2007 10:26:49 -0800 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: pass (herse.apache.org: local policy) Received: from [66.111.4.28] (HELO out4.smtp.messagingengine.com) (66.111.4.28) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 03 Feb 2007 10:26:39 -0800 Received: from out1.internal (unknown [10.202.2.149]) by out1.messagingengine.com (Postfix) with ESMTP id 7DFED16CBB7 for ; Sat, 3 Feb 2007 13:02:37 -0500 (EST) Received: from heartbeat2.messagingengine.com ([10.202.2.161]) by out1.internal (MEProxy); Sat, 03 Feb 2007 13:02:37 -0500 X-Sasl-enc: xeF3fkU7YbEnjkTZIpO1QyMNsfrxbSk+bJMmX0ACof48 1170525756 Received: from [10.17.4.90] (pool-71-162-112-251.bstnma.fios.verizon.net [71.162.112.251]) by mail.messagingengine.com (Postfix) with ESMTP id DEA2E10A1A for ; Sat, 3 Feb 2007 13:02:36 -0500 (EST) Message-ID: <45C4CE35.8030104@mikemccandless.com> Date: Sat, 03 Feb 2007 13:02:29 -0500 From: Michael McCandless User-Agent: Thunderbird 1.5.0.9 (Windows/20061207) MIME-Version: 1.0 To: java-dev@lucene.apache.org Subject: Re: deprecate Directory.renameFile()? References: <200702031739.34400@danielnaber.de> In-Reply-To: <200702031739.34400@danielnaber.de> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org Daniel Naber wrote: > I see that Directory.renameFile() isn't used anymore. I assume it has only > been public for technical reasons, not because we expect this to be used > from outside of Lucene? Should we deprecate this method? Its > implementation e.g. in FSDirectory looks a bit scary anyway (the comment > correctly says "This is not atomic" while the abstract class says "This > replacement should be atomic"). Oh, good catch! I agree it should be deprecated. In addition to the non-atomic issue, that method is also problematic on Windows (intermittent access denied errors, LUCENE-665). Let's deprecate it before 2.1? Mike --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org For additional commands, e-mail: java-dev-help@lucene.apache.org