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

Íñigo Goiri commented on HDFS-12882:
------------------------------------

[~chris.douglas], thanks for [^HDFS-12882.00.salient.txt]. It makes the review 
much easier (we should do this more often :)).
My only concern is changing {{getFileInfo(String src)}} into 
{{getFileInfo(String src, boolean needLocation)}} for {{ClientProtocol}}.
How incompatible is this? I guess targetting 3.1 makes this change OK.
Another option would be to keep the old method and have two paths for the same; 
I don't like this path though, I would prefer to fully do the change.

What about other projects like MapReduce, any breaks there?

> 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-12882.00.salient.txt
>
>
> 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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to