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

Chris Nauroth commented on HADOOP-11713:
----------------------------------------

Hi [~rakeshr].  Thank you for picking up this issue.  Here are a few comments.
# You'll also need to change classes {{ChRootedFs}} and {{ChRootedFileSystem}}.
# Please add tests to {{ViewFsBaseTest}} and {{ViewFileSystemBaseTest}} to 
check the behavior of the various {{throw readOnlyMountTable...}} lines.
# Please add tests to {{TestChRootedFs}} and {{TestChRootedFileSystem}} to 
verify that the chroot jail translates the paths correctly for the new APIs.

HDFS-5594 and HADOOP-10845 are prior patches that demonstrate the basic recipe 
for adding new viewfs methods.  You could look at those if you'd like to get 
some additional ideas about how to write these tests.

Thanks again!

> ViewFileSystem should support snapshot methods.
> -----------------------------------------------
>
>                 Key: HADOOP-11713
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11713
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>    Affects Versions: 3.0.0, 2.2.0
>            Reporter: Chris Nauroth
>            Assignee: Rakesh R
>         Attachments: HDFS-5641-001.patch
>
>
> Currently, {{ViewFileSystem}} does not dispatch snapshot methods through the 
> mount table.  All snapshot methods throw {{UnsupportedOperationException}}, 
> even though the underlying mount points could be HDFS instances that support 
> snapshots.  We need to update {{ViewFileSystem}} to implement the snapshot 
> methods.



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

Reply via email to