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

Andrew Wang commented on HDFS-6509:
-----------------------------------

Hi Charles, quick review comments, might have a few more but overall looks good:

* Would like if FSDirectory#resolvePath, we got rid of the last two ternaries 
as well
* Can we move verifyNotEquals to DFSTestUtil also?
* Could we assert that the paths we get back in the file status are the path we 
passed in? i.e. if stat /.reserved/raw/foo, we get the same thing back from 
stat.getPath()?
* Would be good to do a userspace ls -R test, to make sure that said 
/.reserved/raw prefix is preserved as we recurse

> create a /.reserved/raw filesystem namespace
> --------------------------------------------
>
>                 Key: HDFS-6509
>                 URL: https://issues.apache.org/jira/browse/HDFS-6509
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: security
>    Affects Versions: fs-encryption (HADOOP-10150 and HDFS-6134)
>            Reporter: Charles Lamb
>            Assignee: Charles Lamb
>         Attachments: HDFS-6509.001.patch, HDFS-6509.002.patch, 
> HDFS-6509distcpandDataatRestEncryption-2.pdf, 
> HDFS-6509distcpandDataatRestEncryption-3.pdf, 
> HDFS-6509distcpandDataatRestEncryption.pdf
>
>
> This is part of the work for making distcp work with Data at Rest Encryption. 
> Per the attached document, create a /.reserved/raw HDFS filesystem namespace 
> that allows access to the encrypted bytes of a file.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to