[jira] [Commented] (SAMZA-288) Create config option to disable kill-job-over-rest

["Chris Riccomini (JIRA)" <ji...@apache.org>]

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

Chris Riccomini commented on SAMZA-288:
---------------------------------------

This is actually a very very old feature that used to have a UI link in the AM dashboard. The feature was required because YARN didn't used to have a kill-job script, which they now include. The UI link was removed, but this code was never cleaned up. It seems more YARN-ish to just remove this code entirely and manage job killing through YARN directly.

> Create config option to disable kill-job-over-rest
> --------------------------------------------------
>
>                 Key: SAMZA-288
>                 URL: https://issues.apache.org/jira/browse/SAMZA-288
>             Project: Samza
>          Issue Type: Improvement
>          Components: yarn
>    Affects Versions: 0.7.0
>            Reporter: Jakob Homan
>             Fix For: 0.8.0
>
>
> Currently we expose a rest servlet to kill an AM:
> {code:title=ApplicationMasterRestServlet.scala}  get("/am/kill") {
>     client.kill(state.appAttemptId.getApplicationId)
>   }{code}
> This is useful in some cases, but is also potentially dangerous in some environments.  This feature should be toggled on or off via a config and probably defaulted to off.



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