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

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

Thanks [~vinayrpet] for review..

bq. 2. Instead of directly replacing the current assertions with fully 
qualified path, better to add separate assertions with full qualified path. Let 
current assertions be as is.

I feel this is not required. We have three statement for verifing 
{{setStoragePolicy}}. I change only one statement for fully qualified path, 
other two statement still using path without schema. 


Attached v3 patch and fixed other review comment.

Please review..

> 'storagepolicies -getStoragePolicy' command should accept URI based path.
> -------------------------------------------------------------------------
>
>                 Key: HDFS-11177
>                 URL: https://issues.apache.org/jira/browse/HDFS-11177
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: shell
>    Affects Versions: 2.7.1
>            Reporter: Surendra Singh Lilhore
>            Assignee: Surendra Singh Lilhore
>         Attachments: HDFS-11177.001.patch, HDFS-11177.002.patch, 
> HDFS-11177.003.patch
>
>
> {noformat}
> hdfs storagepolicies -getStoragePolicy -path hdfs://127.0.0.1:8020/t1
> RemoteException: Invalid path name Invalid file name: hdfs://127.0.0.1:8020/t1
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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