[
https://issues.apache.org/jira/browse/RANGER-1918?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
peng.jianhua resolved RANGER-1918.
----------------------------------
Resolution: Not A Problem
> Optimize logic and remove unnecessary temporary variables in YarnClient.class
> -----------------------------------------------------------------------------
>
> Key: RANGER-1918
> URL: https://issues.apache.org/jira/browse/RANGER-1918
> Project: Ranger
> Issue Type: Improvement
> Components: plugins
> Affects Versions: master
> Reporter: peng.jianhua
> Assignee: peng.jianhua
> Fix For: master
>
>
> 1.We have already determined the logic of the response! = Null and
> response.getStatus ()! = 200 in the getQueueResponse method, and do not need
> to be evaluated in the run method of the getQueueResponse method. Move
> "response.close ();" into the getQueueResponse method.
> 2.There is no necessary to create so many useless temporary variables "String
> errMsg = errMessage;"
> just use errMessage is better practice.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
