Return-Path: Delivered-To: apmail-hadoop-mapreduce-issues-archive@minotaur.apache.org Received: (qmail 16481 invoked from network); 1 Oct 2009 05:39:48 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 1 Oct 2009 05:39:48 -0000 Received: (qmail 82603 invoked by uid 500); 1 Oct 2009 04:39:47 -0000 Delivered-To: apmail-hadoop-mapreduce-issues-archive@hadoop.apache.org Received: (qmail 82538 invoked by uid 500); 1 Oct 2009 04:39:47 -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 82521 invoked by uid 99); 1 Oct 2009 04:39:47 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 01 Oct 2009 04:39:47 +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.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 01 Oct 2009 04:39:44 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 88565234C004 for ; Wed, 30 Sep 2009 21:39:23 -0700 (PDT) Message-ID: <1234689218.1254371963544.JavaMail.jira@brutus> Date: Wed, 30 Sep 2009 21:39:23 -0700 (PDT) From: "Jothi Padmanabhan (JIRA)" To: mapreduce-issues@hadoop.apache.org Subject: [jira] Commented: (MAPREDUCE-947) OutputCommitter should have an abortJob method 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-947?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12761093#action_12761093 ] Jothi Padmanabhan commented on MAPREDUCE-947: --------------------------------------------- # Should not the _DONE directory be created only for successful jobs? In the patch, the default implementation of abortJob calls cleanupJob, so _DONE will be created for failed/killed job also, no? # The newly added file Utils should be called something like OutputFilterUtils or something similar # There are unused imports in the test cases # The variable LOG in TestJobCleanup is not used # Do we really need a delayed setup for this test case? Could not we achieve the same using a blocking mapper and then kill/fail the job? In such case, we may not need to rely on signaling the setup job for completion. # The test case should also test for absence of _DONE folder for failed/killed jobs > OutputCommitter should have an abortJob method > ---------------------------------------------- > > Key: MAPREDUCE-947 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-947 > Project: Hadoop Map/Reduce > Issue Type: Improvement > Reporter: Owen O'Malley > Assignee: Amar Kamat > Fix For: 0.21.0 > > Attachments: mapred-948-v1.2.patch, mapred-948-v1.3.patch > > > The OutputCommitter needs an abortJob method to clean up from failed jobs. Currently there is no way to distinguish between failed or succeeded jobs, making it impossible to write output promotion code. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.