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

Yuanbo Liu edited comment on HDFS-10756 at 10/24/16 2:22 PM:
-------------------------------------------------------------

[~xiaochen] Sorry for the late response.
Since HDFS-10883 has been done, I provide v3 patch for this issue according to 
your last comment and my v2 patch.
Some statement:
1) Adding EZ test case of WebHDFS in {{TestEncryptionZones}}
2) Adding EZ test case of HttpFS in {{TestHttpFSServer}} instead of 
{{BaseTestHttpFSWith}}, it's much more easier to test EZ there.
3) Getting rid of some check style bugs in {{BaseTestHttpFSWith}}. It's 
annoying to see those bugs :(.


was (Author: yuanbo):
[~xiaochen] Sorry for the late response.
Since HDFS-10883 has been done, I provide v3 patch for this issue according to 
your last comment and my v2 patch.
Some statement:
1) Adding EZ test case of WebHDFS in {{TestEncryptionZones}}
2) Adding EZ test case of HttpFS in {{TestHttpFSServer}} instead of 
{{BaseTestHttpFSWith}}, it's much more easier to test EZ there.
3) Getting rid of some check style bugs in {{BaseTestHttpFSWith}}. It's 
annoying to see those bugs :).

> Expose getTrashRoot to HTTPFS and WebHDFS
> -----------------------------------------
>
>                 Key: HDFS-10756
>                 URL: https://issues.apache.org/jira/browse/HDFS-10756
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: encryption, httpfs, webhdfs
>            Reporter: Xiao Chen
>            Assignee: Yuanbo Liu
>         Attachments: HDFS-10756.001.patch, HDFS-10756.002.patch, 
> HDFS-10756.003.patch
>
>
> Currently, hadoop FileSystem API has 
> [getTrashRoot|https://github.com/apache/hadoop/blob/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java#L2708]
>  to determine trash directory at run time. Default trash dir is under 
> {{/user/$USER}}
> For an encrypted file, since moving files between/in/out of EZs are not 
> allowed, when an EZ file is deleted via CLI, it calls in to [DFS 
> implementation|https://github.com/apache/hadoop/blob/trunk/hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DistributedFileSystem.java#L2485]
>  to move the file to a trash directory under the same EZ.
> This works perfectly fine for CLI users or java users who call FileSystem 
> API. But for users via httpfs/webhdfs, currently there is no way to figure 
> out what the trash root would be. This jira is proposing we add such 
> interface to httpfs and webhdfs.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to