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

Vinayakumar B commented on HDFS-9337:
-------------------------------------

Looks like validation done for GETDELEGATIONTOKEN are not required. In-fact all 
the params for GETDELEGATIONTOKEN are optional. 

May be need to update the doc for this as well.

[~jagadesh.kiran], thanks for the multiple updates on the patch.
Please remove validation for GETDELEGATIONTOKEN and all related changes in test 
files.
And make all params optional in doc for GETDELEGATIONTOKEN.
Hopefully last update on patch.

Remaining validations looks good.

> Should check required params in WebHDFS to avoid NPE
> ----------------------------------------------------
>
>                 Key: HDFS-9337
>                 URL: https://issues.apache.org/jira/browse/HDFS-9337
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Jagadesh Kiran N
>            Assignee: Jagadesh Kiran N
>         Attachments: HDFS-9337_00.patch, HDFS-9337_01.patch, 
> HDFS-9337_02.patch, HDFS-9337_03.patch, HDFS-9337_04.patch, 
> HDFS-9337_05.patch, HDFS-9337_06.patch, HDFS-9337_07.patch, 
> HDFS-9337_08.patch, HDFS-9337_09.patch, HDFS-9337_10.patch, 
> HDFS-9337_11.patch, HDFS-9337_12.patch, HDFS-9337_13.patch, 
> HDFS-9337_14.patch, HDFS-9337_15.patch, HDFS-9337_16.patch, 
> HDFS-9337_17.patch, HDFS-9337_18.patch
>
>
> {code}
>  curl -i -X PUT 
> "http://10.19.92.127:50070/webhdfs/v1/kiran/sreenu?op=RENAMESNAPSHOT&snapshotname=SNAPSHOTNAME";
> {code}
> Null point exception will be thrown
> {code}
> {"RemoteException":{"exception":"NullPointerException","javaClassName":"java.lang.NullPointerException","message":null}}
> {code}



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

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