[ https://issues.apache.org/jira/browse/YARN-3713?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14566009#comment-14566009 ]
Hudson commented on YARN-3713: ------------------------------ SUCCESS: Integrated in Hadoop-Hdfs-trunk #2141 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk/2141/]) YARN-3713. Remove duplicate function call storeContainerDiagnostics in ContainerDiagnosticsUpdateTransition (zxu via rkanter) (rkanter: rev 6aec13cb338b0fe62ca915f78aa729c9b0b86fba) * hadoop-yarn-project/CHANGES.txt * hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/container/ContainerImpl.java > Remove duplicate function call storeContainerDiagnostics in > ContainerDiagnosticsUpdateTransition > ------------------------------------------------------------------------------------------------ > > Key: YARN-3713 > URL: https://issues.apache.org/jira/browse/YARN-3713 > Project: Hadoop YARN > Issue Type: Improvement > Components: nodemanager > Affects Versions: 2.7.0 > Reporter: zhihai xu > Assignee: zhihai xu > Priority: Minor > Labels: cleanup > Fix For: 2.8.0 > > Attachments: YARN-3713.000.patch > > > remove duplicate function call {{storeContainerDiagnostics}} in > ContainerDiagnosticsUpdateTransition. {{storeContainerDiagnostics}} is > already called at ContainerImpl#addDiagnostics. > {code} > private void addDiagnostics(String... diags) { > for (String s : diags) { > this.diagnostics.append(s); > } > try { > stateStore.storeContainerDiagnostics(containerId, diagnostics); > } catch (IOException e) { > LOG.warn("Unable to update diagnostics in state store for " > + containerId, e); > } > } > {code} > So we don't need call {{storeContainerDiagnostics}} in > ContainerDiagnosticsUpdateTransition#transition. > {code} > container.addDiagnostics(updateEvent.getDiagnosticsUpdate(), "\n"); > try { > container.stateStore.storeContainerDiagnostics(container.containerId, > container.diagnostics); > } catch (IOException e) { > LOG.warn("Unable to update state store diagnostics for " > + container.containerId, e); > } > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)