Ray Chiang commented on YARN-3182:

RE: No new tests

Just a one-line functional equivalent fix.  Mainly a preventative code measure. 
 Found via static code analysis.

> Cleanup switch statement in ApplicationMasterLauncher#handle()
> --------------------------------------------------------------
>                 Key: YARN-3182
>                 URL: https://issues.apache.org/jira/browse/YARN-3182
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Ray Chiang
>            Assignee: Ray Chiang
>            Priority: Minor
>         Attachments: YARN-3182.001.patch
> The last case in the switch statement relies on the break coming from the 
> default case instead of having it's own break.  It's a bit dangerous for any 
> future code modifications in this section.

This message was sent by Atlassian JIRA

Reply via email to