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

Chris Douglas updated HDFS-12882:
---------------------------------
    Attachment: HDFS-12882.00.patch

First pass of the implementation. This enables and extends the contract tests 
added in HDFS-7878. It does the constraint enforcement on the client by 
checking {{HdfsFileStatus}} metadata saved in the {{PathHandle}} against the 
status returned by NN.

The patch adds {{DFSClient::open(HdfsPathHandle, ...)}} that uses an extended 
{{getFileInfo}} call requesting block locations in the response, rather than 
calling {{getBlockLocations}}. Most of the changes are plumbing the 
{{needLocation}} flag through the NN and through the unit tests. By including 
an overloads, as in {{NameNodeAdapter::getFileInfo}}, the patch could be 
smaller, but that doesn't seem consistent with the intent of the class.

Before commit, this needs to verify that the audit log is still correct i.e., 
that calls to {{open}} are not logged as {{getFileStatus}} calls.

> Support full open(PathHandle) contract in HDFS
> ----------------------------------------------
>
>                 Key: HDFS-12882
>                 URL: https://issues.apache.org/jira/browse/HDFS-12882
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: hdfs-client
>            Reporter: Chris Douglas
>            Assignee: Chris Douglas
>         Attachments: HDFS-12882.00.patch
>
>
> HDFS-7878 added support for {{open(PathHandle)}}, but it only partially 
> implemented the semantics specified in the contract (i.e., open-by-inodeID). 
> HDFS should implement all permutations of the default options for 
> {{PathHandle}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to