[
https://issues.apache.org/jira/browse/YARN-4371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15094102#comment-15094102
]
Rohith Sharma K S commented on YARN-4371:
-----------------------------------------
Looking into discussion,
# I agree to [~jlowe]'s comment that number of argument check should be made
very specific to kill. With patch, user can combine with other sub commands. I
missed this part:-(
# As per Jason Lowe's very fist comment, patch keeps Linux behavior that
application id's are space separated values rather then comma separated values.
I think it should be fine.
# About the log message, existing method {{killApplication(applicationId);}}
prints useful message whether application is exist OR does't exist. IMO, newly
added code need not to print this message again. Thoughts?
# With point 3rd assumption, The below code is handled to make sure to keep
Linux behavior of return code. Catching application is just to make sure to
continue with other applications.
{code}
catch (ApplicationNotFoundException e) {
// Suppress all ApplicationNotFoundException for now.
continue;
}
}
if (reportFailure){throw new ApplicationNotFoundException("Application doesn't
exist in RM.");}{code}
I.e when killing multiple process and if one process is success then return
code is zero.
{code}
root1@root1-ThinkPad-T440p:~/workspace/project_home/hadoop-3.0.0-SNAPSHOT/bin$
kill 24187 12345
bash: kill: (12345) - No such process
root1@root1-ThinkPad-T440p:~/workspace/project_home/hadoop-3.0.0-SNAPSHOT/bin$
echo $?
0
{code}
> "yarn application -kill" should take multiple application ids
> -------------------------------------------------------------
>
> Key: YARN-4371
> URL: https://issues.apache.org/jira/browse/YARN-4371
> Project: Hadoop YARN
> Issue Type: Improvement
> Reporter: Tsuyoshi Ozawa
> Assignee: Sunil G
> Attachments: 0001-YARN-4371.patch, 0002-YARN-4371.patch,
> 0003-YARN-4371.patch
>
>
> Currently we cannot pass multiple applications to "yarn application -kill"
> command. The command should take multiple application ids at the same time.
> Each entries should be separated with whitespace like:
> {code}
> yarn application -kill application_1234_0001 application_1234_0007
> application_1234_0012
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
