Jason Lowe commented on YARN-3619:

Thanks for updating the patch!  In the future, please do not remove patches 
unless absolutely necessary as it generates a lot of email churn and can lead 
to confusion as in this case.  The patchbot comments above are _not_ for the 
most recent patch applied despite the patch basename being the same.  This 
subsequent patch should have simply been version 002.

Latest patch looks good to me, checkstyle and release audit warnings were 
pre-existing.  Kicking Jenkins again to comment on the latest patch.

> ContainerMetrics unregisters during getMetrics and leads to 
> ConcurrentModificationException
> -------------------------------------------------------------------------------------------
>                 Key: YARN-3619
>                 URL: https://issues.apache.org/jira/browse/YARN-3619
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>    Affects Versions: 2.7.0
>            Reporter: Jason Lowe
>            Assignee: zhihai xu
>         Attachments: YARN-3619.000.patch, YARN-3619.001.patch, test.patch
> ContainerMetrics is able to unregister itself during the getMetrics method, 
> but that method can be called by MetricsSystemImpl.sampleMetrics which is 
> trying to iterate the sources.  This leads to a 
> ConcurrentModificationException log like this:
> {noformat}
> 2015-05-11 14:00:20,360 [Timer for 'NodeManager' metrics system] WARN 
> impl.MetricsSystemImpl: java.util.ConcurrentModificationException
> {noformat}

This message was sent by Atlassian JIRA

Reply via email to