hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rakesh R (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-8495) Consolidate append() related implementation into a single class
Date Mon, 06 Jul 2015 11:33:05 GMT

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

Rakesh R commented on HDFS-8495:
--------------------------------

Thanks a lot [~vinayrpet] for the reviews.

{{FSDirAppendOp.prepareFileForAppend()}} function is used by {{FSEditLogLoader#applyEditLogOp}}
[Line number # 338|https://github.com/apache/hadoop/blob/trunk/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/FSEditLogLoader.java#L338]
I could see here it is resolving the path in a different way not directly taking from {{iip.getPath()}}.

{code}
      final String path =
          renameReservedPathsOnUpgrade(addCloseOp.path, logVersion);
{code}

> Consolidate append() related implementation into a single class
> ---------------------------------------------------------------
>
>                 Key: HDFS-8495
>                 URL: https://issues.apache.org/jira/browse/HDFS-8495
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: namenode
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>         Attachments: HDFS-8495-000.patch, HDFS-8495-001.patch, HDFS-8495-002.patch, HDFS-8495-003.patch,
HDFS-8495-003.patch, HDFS-8495-004.patch
>
>
> This jira proposes to consolidate {{FSNamesystem#append()}} related methods into a single
class.



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

Mime
View raw message