Return-Path: Delivered-To: apmail-hadoop-hdfs-commits-archive@minotaur.apache.org Received: (qmail 66776 invoked from network); 19 Oct 2009 21:53:09 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 19 Oct 2009 21:53:09 -0000 Received: (qmail 48954 invoked by uid 500); 19 Oct 2009 21:53:09 -0000 Delivered-To: apmail-hadoop-hdfs-commits-archive@hadoop.apache.org Received: (qmail 48904 invoked by uid 500); 19 Oct 2009 21:53:09 -0000 Mailing-List: contact hdfs-commits-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hdfs-dev@hadoop.apache.org Delivered-To: mailing list hdfs-commits@hadoop.apache.org Received: (qmail 48893 invoked by uid 99); 19 Oct 2009 21:53:09 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 19 Oct 2009 21:53:09 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 19 Oct 2009 21:53:00 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 260D4238889B; Mon, 19 Oct 2009 21:52:39 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r826823 - in /hadoop/hdfs/trunk: CHANGES.txt src/test/aop/org/apache/hadoop/hdfs/HFlushAspects.aj Date: Mon, 19 Oct 2009 21:52:39 -0000 To: hdfs-commits@hadoop.apache.org From: cos@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20091019215239.260D4238889B@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: cos Date: Mon Oct 19 21:52:38 2009 New Revision: 826823 URL: http://svn.apache.org/viewvc?rev=826823&view=rev Log: HDFS-713. Need to properly check the type of the test class from an aspect. Contributed by Konstantin Boudnik Modified: hadoop/hdfs/trunk/CHANGES.txt hadoop/hdfs/trunk/src/test/aop/org/apache/hadoop/hdfs/HFlushAspects.aj Modified: hadoop/hdfs/trunk/CHANGES.txt URL: http://svn.apache.org/viewvc/hadoop/hdfs/trunk/CHANGES.txt?rev=826823&r1=826822&r2=826823&view=diff ============================================================================== --- hadoop/hdfs/trunk/CHANGES.txt (original) +++ hadoop/hdfs/trunk/CHANGES.txt Mon Oct 19 21:52:38 2009 @@ -288,6 +288,9 @@ tests and changed SleepAction to support uniform random sleeping over an interval. (szetszwo) + HDFS-713. Need to properly check the type of the test class from an aspect + (cos) + BUG FIXES HDFS-76. Better error message to users when commands fail because of Modified: hadoop/hdfs/trunk/src/test/aop/org/apache/hadoop/hdfs/HFlushAspects.aj URL: http://svn.apache.org/viewvc/hadoop/hdfs/trunk/src/test/aop/org/apache/hadoop/hdfs/HFlushAspects.aj?rev=826823&r1=826822&r2=826823&view=diff ============================================================================== --- hadoop/hdfs/trunk/src/test/aop/org/apache/hadoop/hdfs/HFlushAspects.aj (original) +++ hadoop/hdfs/trunk/src/test/aop/org/apache/hadoop/hdfs/HFlushAspects.aj Mon Oct 19 21:52:38 2009 @@ -23,6 +23,7 @@ import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.apache.hadoop.fi.DataTransferTestUtil; +import org.apache.hadoop.fi.PipelineTest; import org.apache.hadoop.fi.FiHFlushTestUtil.HFlushTest; import org.apache.hadoop.hdfs.DFSClient.DFSOutputStream; import org.apache.hadoop.hdfs.protocol.DatanodeInfo; @@ -46,12 +47,14 @@ LOG.info("No pipeline is built"); return; } - if (DataTransferTestUtil.getPipelineTest() == null) { + PipelineTest pt = DataTransferTestUtil.getPipelineTest(); + if (pt == null) { LOG.info("No test has been initialized"); return; } - for (int i=0; i