[
https://issues.apache.org/jira/browse/HADOOP-7704?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Devaraj K moved MAPREDUCE-3129 to HADOOP-7704:
----------------------------------------------
Component/s: (was: mrv2)
Fix Version/s: (was: 0.24.0)
0.24.0
Affects Version/s: (was: 0.24.0)
0.24.0
Key: HADOOP-7704 (was: MAPREDUCE-3129)
Project: Hadoop Common (was: Hadoop Map/Reduce)
> JsonFactory can be created only once and used for every next request to
> create JsonGenerator inside JMXJsonServlet
> -------------------------------------------------------------------------------------------------------------------
>
> Key: HADOOP-7704
> URL: https://issues.apache.org/jira/browse/HADOOP-7704
> Project: Hadoop Common
> Issue Type: Bug
> Affects Versions: 0.24.0
> Reporter: Devaraj K
> Assignee: Devaraj K
> Priority: Minor
> Fix For: 0.24.0
>
> Attachments: MAPREDUCE-3129.patch
>
>
> 1. Currently JMXJsonServlet creates JsonFactory for every http request.
> Its not efficient.
> JsonFactory can be created only once and used for every next request to
> create JsonGenerator.
> 2. Also following null check is not required.
> {code}
> if (mBeanServer == null) {
> jg.writeStringField("result", "ERROR");
> jg.writeStringField("message", "No MBeanServer could be found");
> jg.close();
> LOG.error("No MBeanServer could be found.");
> response.setStatus(HttpServletResponse.SC_NOT_FOUND);
> return;
> }
> {code}
> Because ManagementFactory.getPlatformMBeanServer(); will not return null.
> 3. Move the following code to finally so that any exception should not cause
> skipping of close method on JsonGenerator
> {code}
> jg.close();
> {code}
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
