hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tsz Wo (Nicholas), SZE (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HDFS-570) When opening a file for read, make the file length avaliable to client.
Date Wed, 02 Sep 2009 21:01:34 GMT

    [ https://issues.apache.org/jira/browse/HDFS-570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12750663#action_12750663

Tsz Wo (Nicholas), SZE commented on HDFS-570:

> Could you explain more?

As I said before, you mix in your patch changes related to the current jira with unrelated
refactoring of the code.
I listed most of the cases (1-8) that do not belong to the functionality you are implementing.
And which

   1. Obscure understanding of the new functionality you actually introduce.
   2. Make it hard to continue merging the trunk with the branch.

My proposal is to separate the implementation of the visible length from the refactoring of
the code into 2 separate patches.
The refactoring should be applied then to the trunk and to the branch.
My personal preference is to postpone the refactoring until append is merged to the trunk.
There is no code refactoring in LocatedBlock except for the imports.  What do you mean by
"LocatedBlock 3. Does not need any of the changes."?

getReplicaInfo() is currently a private method of FSDataset. You are adding it to FSDatasetInterface.
Based on the usage of the method I don't see a need for that.
If you look at the patch again, you will find that getReplicaInfo(..) is called in DataNode.
 It cannot be invoked if it is a private method of FSDataset.

> When opening a file for read, make the file length avaliable to client.
> -----------------------------------------------------------------------
>                 Key: HDFS-570
>                 URL: https://issues.apache.org/jira/browse/HDFS-570
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs client
>    Affects Versions: Append Branch
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Tsz Wo (Nicholas), SZE
>             Fix For: Append Branch
>         Attachments: h570_20090828.patch
> In order to support read consistency, DFSClient needs the file length at the file opening
time.  In the current implmentation, DFSClient obtains the file length at the file opening
time but the length is inaccurate if the file is being written.
> For more details, see Section 4 in the [append design doc|https://issues.apache.org/jira/secure/attachment/12415768/appendDesign2.pdf].

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

View raw message