[ https://issues.apache.org/jira/browse/YARN-3628?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14541368#comment-14541368 ]
zhihai xu commented on YARN-3628: --------------------------------- I updated the patch to fix the whitespace issue. > ContainerMetrics should support always-flush mode. > -------------------------------------------------- > > Key: YARN-3628 > URL: https://issues.apache.org/jira/browse/YARN-3628 > Project: Hadoop YARN > Issue Type: Bug > Reporter: zhihai xu > Assignee: zhihai xu > Attachments: YARN-3628.000.patch > > > ContainerMetrics should support always-flush mode. > It will be good to set ContainerMetrics as always-flush mode if > yarn.nodemanager.container-metrics.period-ms is configured as 0. > Currently both 0 and -1 mean flush on completion. > Also the current default value for > yarn.nodemanager.container-metrics.period-ms is -1 and the default value for > yarn.nodemanager.container-metrics.enable is true. So the empty content is > shown for the active container metrics until it is finished. > The default value for yarn.nodemanager.container-metrics.period-ms should not > be -1. > flushOnPeriod is always false if flushPeriodMs is -1, > the content will only be shown when the container is finished. > {code} > if (finished || flushOnPeriod) { > registry.snapshot(collector.addRecord(registry.info()), all); > } > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)