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

Surendra Singh Lilhore commented on HDFS-13292:
-----------------------------------------------

Thanks for the patch [~RANith] and  thanks [~shahrs87] for discussion.
{quote}The only downside of throwing {{not an empty directory}} exception first 
is some user might delete the contents of directory and not realize it is 
already in an EZ.
{quote}
Same case I am thinking. I feel we should fix this. Client should get the clear 
info from exception.

> Crypto command should give proper exception when key is already exist for 
> zone directory
> ----------------------------------------------------------------------------------------
>
>                 Key: HDFS-13292
>                 URL: https://issues.apache.org/jira/browse/HDFS-13292
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: hdfs, kms
>    Affects Versions: 2.8.3
>            Reporter: Harshakiran Reddy
>            Priority: Major
>         Attachments: HDFS-13292.001.patch
>
>
> {{Scenario:}}
>  # Create a Dir
>  # Create EZ for the above dir with Key1
>  # Again you can try to create ZONE for same directory with Diff Key i.e Key2
> {noformat}
> hadoopclient> hadoop key list
> Listing keys for KeyProvider: 
> org.apache.hadoop.crypto.key.kms.LoadBalancingKMSClientProvider@152aa092
> key2
> key1
> hadoopclient> hdfs dfs -mkdir /kms
> hadoopclient> hdfs dfs -put bigdata_env /kms/file1
> hadoopclient> hdfs crypto -createZone -keyName key1 -path /kms
> RemoteException: Attempt to create an encryption zone for a non-empty 
> directory.
> hadoopclient> hdfs dfs -rmr /kms/file1
> rmr: DEPRECATED: Please use '-rm -r' instead.
> Deleted /kms/file1
> hadoopclient> hdfs crypto -createZone -keyName key1 -path /kms
> Added encryption zone /kms
> hadoopclient> hdfs crypto -createZone -keyName key2 -path /kms
> RemoteException: Attempt to create an encryption zone for a non-empty 
> directory.
> hadoopclient>
>  {noformat}
> Actual Output:
> ===========
> {{Exception should be Like Dir already having the ZONE will not allow to 
> create new ZONE on this Dir}}
> Expected Output:
> =============
> {{RemoteException:Attempt to create an encryption zone for non-empty 
> directory}}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
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