[ 
https://issues.apache.org/jira/browse/HDFS-624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12758821#action_12758821
 ] 

Suresh Srinivas commented on HDFS-624:
--------------------------------------

Comments:
# getNewStampForPipeline - is it better to rename this as getNewGenerationStamp?
# DFSClient.java - ResponseProcessor.run() - To avoid a warning in style check 
for empty block, can the if statement be changed to {{ if (seqno != -2) }}
# ResponseProcessor.run() - In the for loop before throwing exception should 
{{responderClosed = true}} be set?
# ResponseProcessor.run() - why is the try - catch block not inside the for 
loop? Is setting errorIndex to 0 fine?

> Client support pipeline recovery
> --------------------------------
>
>                 Key: HDFS-624
>                 URL: https://issues.apache.org/jira/browse/HDFS-624
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>    Affects Versions: Append Branch
>            Reporter: Hairong Kuang
>            Assignee: Hairong Kuang
>             Fix For: Append Branch
>
>         Attachments: pipelineRecovery.patch, pipelineRecovery1.patch
>
>
> This jira aims to
> 1. set up initial pipeline for append;
> 2. recover failed pipeline setup for append;
> 2. set up pipeline to recover failed data streaming.
> The algorithm is described in the design document in the pipeline recovery 
> and pipeline set up sections. Pipeline close and failed pipeline close are 
> not included in this jira. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to