peng.jianhua created RANGER-1918:
------------------------------------

             Summary: 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)

Reply via email to