[RANGER-1918] Optimize logic and remove unnecessary temporary variables in YarnClient.class

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Not A Problem
• Affects Version/s: master
• Component/s: plugins
• Flags: Patch
• Assignee: peng.jianhua
• Reporter: peng.jianhua
• Created: 07/Dec/17 05:38
• Updated: 02/Jul/18 20:20
• Resolved: 08/Dec/17 00:47

Description

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.

Issue Links

• links to: https://reviews.apache.org/r/64404/