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

Weiwei Yang commented on YARN-8693:
-----------------------------------

Hi [~Tao Yang]

Thanks for adding this. I have 2 comments regarding the patch.

1. In {{RMWebServices#signalToContainer}}, it throws a server-side exception 
when something goes wrong in handling the request. Can we send a proper 
response to client too? E.g 
{code:java}
return Response.status(Status.INTERNAL_SERVER_ERROR).entity(e).build();
{code}
2. you please update the doc too?

Thanks

> Add signalToContainer REST API for RMWebServices
> ------------------------------------------------
>
>                 Key: YARN-8693
>                 URL: https://issues.apache.org/jira/browse/YARN-8693
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: restapi
>            Reporter: Tao Yang
>            Assignee: Tao Yang
>            Priority: Major
>         Attachments: YARN-8693.001.patch, YARN-8693.002.patch
>
>
> Currently YARN has a RPC command which is "yarn container -signal <container 
> ID [signal command]>" to signal 
> OUTPUT_THREAD_DUMP/GRACEFUL_SHUTDOWN/FORCEFUL_SHUTDOWN commands to container. 
> That is not enough and we need to add signalToContainer REST API for better 
> management from cluster administrators or management system.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to