Rohith Sharma K S commented on YARN-4371:

Going through this JIRA, really helps a lot for killing applications instead of 
killing one by one. Thanks [~sunilg] for working on this JIRA. 

Looking into patch, overall patch approach looks good.
one comment
# Instead of changing existing killAplication(ApplicationId appId) API, 
introduce new api with arrays as an argument as you have done in patch 
{{killApplication(String[] applicationIds)}}. Iterate for each application 
inside this new method. something similar to below 
killApplication(String[] applicationIds){
      for (String applicationId : applicationIds) {
      killApplication(String applicationId);

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

Reply via email to