Return-Path: Delivered-To: apmail-hadoop-hdfs-issues-archive@minotaur.apache.org Received: (qmail 18289 invoked from network); 23 Jun 2010 17:36:15 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 23 Jun 2010 17:36:15 -0000 Received: (qmail 70229 invoked by uid 500); 23 Jun 2010 17:36:15 -0000 Delivered-To: apmail-hadoop-hdfs-issues-archive@hadoop.apache.org Received: (qmail 70147 invoked by uid 500); 23 Jun 2010 17:36:14 -0000 Mailing-List: contact hdfs-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hdfs-issues@hadoop.apache.org Delivered-To: mailing list hdfs-issues@hadoop.apache.org Received: (qmail 70138 invoked by uid 99); 23 Jun 2010 17:36:14 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 23 Jun 2010 17:36:14 +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.22] (HELO thor.apache.org) (140.211.11.22) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 23 Jun 2010 17:36:12 +0000 Received: from thor (localhost [127.0.0.1]) by thor.apache.org (8.13.8+Sun/8.13.8) with ESMTP id o5NHZot3003523 for ; Wed, 23 Jun 2010 17:35:51 GMT Message-ID: <14958335.19501277314550813.JavaMail.jira@thor> Date: Wed, 23 Jun 2010 13:35:50 -0400 (EDT) From: "Todd Lipcon (JIRA)" To: hdfs-issues@hadoop.apache.org Subject: [jira] Commented: (HDFS-1263) 0.20: in tryUpdateBlock, the meta file is renamed away before genstamp validation is done In-Reply-To: <23099098.12041277280650583.JavaMail.jira@thor> 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/HDFS-1263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12881776#action_12881776 ] Todd Lipcon commented on HDFS-1263: ----------------------------------- bq. also, we check the genstamp is moving upwards both at the start of updateBlock and at the end of tryUpdateBlock. do you know why? Oops, missed this question - probably because synchronization blocks are dropped and reclaimed in every cycle of checking that no threads are writing the block. But we should certainly do the sanity check at the top of the function instead of after renaming the meta file > 0.20: in tryUpdateBlock, the meta file is renamed away before genstamp validation is done > ----------------------------------------------------------------------------------------- > > Key: HDFS-1263 > URL: https://issues.apache.org/jira/browse/HDFS-1263 > Project: Hadoop HDFS > Issue Type: Bug > Components: data-node > Affects Versions: 0.20-append > Reporter: Todd Lipcon > Assignee: Todd Lipcon > Fix For: 0.20-append > > > Saw an issue where multiple datanodes are trying to recover at the same time, and all of them failed. I think the issue is in FSDataset.tryUpdateBlock, we do the rename of blk_B_OldGS to blk_B_OldGS_tmpNewGS and *then* check that the generation stamp is moving upwards. Because of this, invalid update block calls are blocked, but they then cause future updateBlock calls to fail with "Meta file not found" errors. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.