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

Rohith Sharma K S commented on YARN-5970:
-----------------------------------------

bq. Since we do not know the error offset, do we need to set it to -1?
offset is mandatory for parseexcpetion. However, since null value is being 
passed, offset remain as -1.

bq. Could we append "seconds" as well at the end.
This value can not be seconds. This value be in-valid ISO8601 format. so, we 
need to print invalid configured value that can be anything. 

bq.  think point 3 is handled in YARN-5965
yes, yarn-5965 handles comment-3.

> Validate update timeout request parameters
> ------------------------------------------
>
>                 Key: YARN-5970
>                 URL: https://issues.apache.org/jira/browse/YARN-5970
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: scheduler
>            Reporter: Rohith Sharma K S
>            Assignee: Rohith Sharma K S
>         Attachments: YARN-5970.0.patch
>
>
> AppTimeoutInfo is exposed to REST clients. This object request parameters 
> should be validated before processing it to clientRMService.
> And also handles couple of minor issues in REST services.



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