Return-Path: X-Original-To: apmail-hadoop-mapreduce-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-mapreduce-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 18FDF765C for ; Wed, 12 Oct 2011 12:03:36 +0000 (UTC) Received: (qmail 42594 invoked by uid 500); 12 Oct 2011 12:03:36 -0000 Delivered-To: apmail-hadoop-mapreduce-issues-archive@hadoop.apache.org Received: (qmail 42401 invoked by uid 500); 12 Oct 2011 12:03:35 -0000 Mailing-List: contact mapreduce-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: mapreduce-issues@hadoop.apache.org Delivered-To: mailing list mapreduce-issues@hadoop.apache.org Received: (qmail 42393 invoked by uid 99); 12 Oct 2011 12:03:35 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 12 Oct 2011 12:03:35 +0000 X-ASF-Spam-Status: No, hits=-2000.5 required=5.0 tests=ALL_TRUSTED,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 12 Oct 2011 12:03:33 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id 3D63B304667 for ; Wed, 12 Oct 2011 12:03:12 +0000 (UTC) Date: Wed, 12 Oct 2011 12:03:12 +0000 (UTC) From: "Ravi Gummadi (Commented) (JIRA)" To: mapreduce-issues@hadoop.apache.org Message-ID: <276241708.4557.1318420992252.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <1713823593.46960.1313620707236.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (MAPREDUCE-2850) TaskTracker disk failure handling (MR-2413) has no test coverage MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/MAPREDUCE-2850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13125749#comment-13125749 ] Ravi Gummadi commented on MAPREDUCE-2850: ----------------------------------------- >> In prepareDirToFail it says the file is set to perms "000" but File#createNewFile uses the default perms (eg 644 with umask 022), so it should still be accessible right? No. The comment was wrong. I replace the directory with a file so that DiskChecker.checkDirs() will fail because it tries to do mkdir with the same name and this will be reported as a disk failure. Updating the comment accordingly. >> If you want not always have waitForDiskHealthCheck wait for 10s at a time seems like you can lower the DISK_HEALTH_CHECK_INTERVAL to eg 1s. OK. Changing to 1 sec. >> Would also be good to test startup with a failed directory. Feel free to punt this to MAPREDUCE-2921. This change would need a handle into the code of MiniMRCluster.TaskTrackerRunner() and needs some refactoring and exposing some api in MiniMRCluster. So not doing it as part of current JIRA. > TaskTracker disk failure handling (MR-2413) has no test coverage > ---------------------------------------------------------------- > > Key: MAPREDUCE-2850 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-2850 > Project: Hadoop Map/Reduce > Issue Type: Sub-task > Components: tasktracker > Affects Versions: 0.20.204.0 > Reporter: Eli Collins > Assignee: Ravi Gummadi > Attachments: MR2850.v0.patch > > > MR-2413 doesn't have any test coverage that eg tests that the TT can survive disk failure. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira