hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yi Liu (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HDFS-7638) Small fix and few refinements for FSN#truncate
Date Mon, 19 Jan 2015 08:30:34 GMT

     [ https://issues.apache.org/jira/browse/HDFS-7638?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Yi Liu updated HDFS-7638:
-------------------------
    Attachment: HDFS-7638.001.patch

> Small fix and few refinements for FSN#truncate
> ----------------------------------------------
>
>                 Key: HDFS-7638
>                 URL: https://issues.apache.org/jira/browse/HDFS-7638
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: datanode, namenode
>            Reporter: Yi Liu
>            Assignee: Yi Liu
>             Fix For: 2.7.0
>
>         Attachments: HDFS-7638.001.patch
>
>
> *1.* 
> {code}
> removeBlocks(collectedBlocks);
> {code}
> should be after {{logSync}}, as we do in other FSN places (rename, delete, write with
overwrite), the reason is discussed in HDFS-2815 and https://issues.apache.org/jira/browse/HDFS-6871?focusedCommentId=14110068&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14110068
> *2.*
> {code}
> stat = FSDirStatAndListingOp.getFileInfo(dir, src, false,
>     FSDirectory.isReservedRawName(src), true);
> {code}
> We'd better to use {{dir.getAuditFileInfo}}, since it's only for audit log. If audit
log is not on, we don't need to get the file info.
> *3.*
> In {{truncateInternal}}, 
> {code}
> INodeFile file = iip.getLastINode().asFile();
> {code}
> is not necessary. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message