Return-Path: Delivered-To: apmail-hadoop-hdfs-issues-archive@minotaur.apache.org Received: (qmail 58538 invoked from network); 28 Sep 2009 18:41:38 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 28 Sep 2009 18:41:38 -0000 Received: (qmail 63505 invoked by uid 500); 28 Sep 2009 18:41:38 -0000 Delivered-To: apmail-hadoop-hdfs-issues-archive@hadoop.apache.org Received: (qmail 63470 invoked by uid 500); 28 Sep 2009 18:41:38 -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 63460 invoked by uid 99); 28 Sep 2009 18:41:38 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 28 Sep 2009 18:41:38 +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; Mon, 28 Sep 2009 18:41:36 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 0F70A234C004 for ; Mon, 28 Sep 2009 11:41:16 -0700 (PDT) Message-ID: <850326287.1254163276048.JavaMail.jira@brutus> Date: Mon, 28 Sep 2009 11:41:16 -0700 (PDT) From: "Suresh Srinivas (JIRA)" To: hdfs-issues@hadoop.apache.org Subject: [jira] Commented: (HDFS-642) Support pipeline close and close recovery In-Reply-To: <425964827.1253657416059.JavaMail.jira@brutus> 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-642?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760330#action_12760330 ] Suresh Srinivas commented on HDFS-642: -------------------------------------- # Some of the changed code goes beyond 80 chars # DFSClient.java - can the following check be made in a separate method {{(!streamerClosed && !hasError && clientRunning)}} # DFSClient.java - waiting for ack (ack size to become zero) can become a separate method # DFSClient readChunk() - {{( dataLen <= 0 && !lastPacketInBlock )}} should the first part be just datalen == 0? # DFSClient.Packet - An optimization that could be made in future is to not to allocate any memory to Packet.buf for zero length packet # Nit: FSDatasetInterface.java - Change comment {and returns the meta info of the replica} to {and return ...} # DFSClient.java not sure why the waiting for ack queue to become zero len was moved out of previous try catch? > Support pipeline close and close recovery > ----------------------------------------- > > Key: HDFS-642 > URL: https://issues.apache.org/jira/browse/HDFS-642 > Project: Hadoop HDFS > Issue Type: Sub-task > Reporter: Hairong Kuang > Assignee: Hairong Kuang > Fix For: Append Branch > > Attachments: closePipeline.patch > > > This jira intends to implement the third stage of block construction: close. The main ideas are: > 1. A client notifies the datanodes in the pipeline to finalize a replica and then shuts down the pipeline by sending a special end-of-block message. > 2. The end-of-block message does not get sent until the client finishes sending all packets belong to the block and have received all their acks. > 3. If close fails, the client recovers the failures by setting up a new pipeline with the remaining datanodes and a newer generation stamp. > 4. I plan to use an empty packet as the end-of-block message. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.