Andrew Wang commented on HDFS-10883:

Thanks for working on this [~yuanbo], just one nit:

* testNestedEncryptionZones, we can just let the exception bubble rather than 
catching and doing an explicit "fail". You can put the fail message as a 

Otherwise looks good, thanks for working on this!

> `getTrashRoot`'s behavior is not consistent in DFS after enabling EZ.
> ---------------------------------------------------------------------
>                 Key: HDFS-10883
>                 URL: https://issues.apache.org/jira/browse/HDFS-10883
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Yuanbo Liu
>            Assignee: Yuanbo Liu
>         Attachments: HDFS-10883-test-case.txt, HDFS-10883.001.patch, 
> HDFS-10883.002.patch, HDFS-10883.003.patch
> Let's say root path ("/") is the encryption zone, and there is a file called 
> "/test" in root path.
> {code}
> dfs.getTrashRoot(new Path("/"))
> {code}
> returns "/user/$USER/.Trash",
> while
> {code}
> dfs.getTrashRoot(new Path("/test"))
> {code} 
> returns "/.Trash/$USER".
> Please see the attachment to know how to reproduce this issue.

This message was sent by Atlassian JIRA

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