Return-Path: Delivered-To: apmail-lucene-hadoop-commits-archive@locus.apache.org Received: (qmail 35696 invoked from network); 21 Nov 2007 16:21:27 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 21 Nov 2007 16:21:27 -0000 Received: (qmail 25625 invoked by uid 500); 21 Nov 2007 16:21:15 -0000 Delivered-To: apmail-lucene-hadoop-commits-archive@lucene.apache.org Received: (qmail 25509 invoked by uid 500); 21 Nov 2007 16:21:14 -0000 Mailing-List: contact hadoop-commits-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hadoop-dev@lucene.apache.org Delivered-To: mailing list hadoop-commits@lucene.apache.org Received: (qmail 25500 invoked by uid 99); 21 Nov 2007 16:21:14 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Nov 2007 08:21:14 -0800 X-ASF-Spam-Status: No, hits=-100.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO eris.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Nov 2007 16:21:12 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id CBCF41A9832; Wed, 21 Nov 2007 08:21:05 -0800 (PST) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r597133 - in /lucene/hadoop/trunk: CHANGES.txt src/java/org/apache/hadoop/fs/ChecksumFileSystem.java Date: Wed, 21 Nov 2007 16:21:05 -0000 To: hadoop-commits@lucene.apache.org From: ddas@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20071121162105.CBCF41A9832@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: ddas Date: Wed Nov 21 08:21:00 2007 New Revision: 597133 URL: http://svn.apache.org/viewvc?rev=597133&view=rev Log: HADOOP-2212. ChecksumFileSystem.getSumBufferSize might throw java.lang.ArithmeticException. The fix is to initialize bytesPerChecksum to 0. Contributed by Michael Bieniosek Modified: lucene/hadoop/trunk/CHANGES.txt lucene/hadoop/trunk/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java Modified: lucene/hadoop/trunk/CHANGES.txt URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/CHANGES.txt?rev=597133&r1=597132&r2=597133&view=diff ============================================================================== --- lucene/hadoop/trunk/CHANGES.txt (original) +++ lucene/hadoop/trunk/CHANGES.txt Wed Nov 21 08:21:00 2007 @@ -122,6 +122,10 @@ completed by another (possibly speculative) map-task. This ensures that map-slots are not needlessly held-up. (acmurthy) + HADOOP-2212. ChecksumFileSystem.getSumBufferSize might throw + java.lang.ArithmeticException. The fix is to initialize bytesPerChecksum + to 0. (Michael Bieniosek via ddas) + Release 0.15.1 - 2007-11-27 Modified: lucene/hadoop/trunk/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java URL: http://svn.apache.org/viewvc/lucene/hadoop/trunk/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java?rev=597133&r1=597132&r2=597133&view=diff ============================================================================== --- lucene/hadoop/trunk/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java (original) +++ lucene/hadoop/trunk/src/java/org/apache/hadoop/fs/ChecksumFileSystem.java Wed Nov 21 08:21:00 2007 @@ -37,7 +37,7 @@ *****************************************************************/ public abstract class ChecksumFileSystem extends FilterFileSystem { private static final byte[] CHECKSUM_VERSION = new byte[] {'c', 'r', 'c', 0}; - private int bytesPerChecksum; + private int bytesPerChecksum = 512; public static double getApproxChkSumLength(long size) { return ChecksumFSOutputSummer.CHKSUM_AS_FRACTION * size;