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

shanyu zhao commented on YARN-1713:
-----------------------------------

[~vvasudev] I tested this patch and it works great!

I thought about your implementation of reflecting how RPC structures the API 
and it sounds good to me. 

The only feedback I have is the style of the REST API, does the following API 
change make sense for you?
POST /app/id --> POST /appids
POST /app/{appid} --> POST /apps/{appid}
This way the submit app REST API is more consistent with app status and kill 
app API.

Sorry I missed the part that your implementation actually calls doAs() on 
behalf of the remote user. That should work just fine. 

Thanks!

> Implement getnewapplication and submitapp as part of RM web service
> -------------------------------------------------------------------
>
>                 Key: YARN-1713
>                 URL: https://issues.apache.org/jira/browse/YARN-1713
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Varun Vasudev
>            Assignee: Varun Vasudev
>         Attachments: apache-yarn-1713.3.patch, apache-yarn-1713.4.patch, 
> apache-yarn-1713.5.patch, apache-yarn-1713.cumulative.2.patch, 
> apache-yarn-1713.cumulative.3.patch, apache-yarn-1713.cumulative.4.patch, 
> apache-yarn-1713.cumulative.patch, apache-yarn-1713.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to