[ 
https://issues.apache.org/jira/browse/HADOOP-9281?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ivan A. Veselovsky updated HADOOP-9281:
---------------------------------------

    Attachment: HADOOP-9281-trunk--N6--MAPREDUCE-part.patch
                HADOOP-9281-trunk--N6--HADOOP-COMMON-part.patch

Hi, Daryn,
please see xxx--N6--yyy patches (apply to both trunk & branch2).
1) I removed "unnecessary" changes from the patch. Some "unnecessary" changes 
were initially done to compensate pre-commit audit warnings (mostly 
deprecations, since the patch deprecates old MetricsServelet), so now the patch 
may have problems with the number of warnings.
2) I splitted the patch into 2 pieces (hadoop-common and mr part). Yarn part is 
"unnecessary". 
3) ServletSink.numInstances was not thread safe, right. It was so , because 
creation of several Servlet instances is quite rare case, though, this is 
possible in theory. I fixed the thread-safety issue.

> Rework all usages of o.a.h.metrics to o.a.h.metrics2 new metrics API
> --------------------------------------------------------------------
>
>                 Key: HADOOP-9281
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9281
>             Project: Hadoop Common
>          Issue Type: Improvement
>    Affects Versions: 3.0.0, 2.0.3-alpha, 0.23.7
>            Reporter: Ivan A. Veselovsky
>            Assignee: Ivan A. Veselovsky
>         Attachments: HADOOP-9281-branch-0.23--N5.patch, 
> HADOOP-9281-trunk--N5.patch, HADOOP-9281-trunk--N6--HADOOP-COMMON-part.patch, 
> HADOOP-9281-trunk--N6--MAPREDUCE-part.patch, HADOOP-9281-trunk--N6.patch
>
>
> The following done:
> 1) o.a.h.metrics.MetricsServlet reworked to o.a.h.metrics2.lib.MetricsServlet2
> 2) class org.apache.hadoop.mapreduce.task.reduce.ShuffleClientMetrics 
> rewritten to use metrics2 API.
> 3) class org.apache.hadoop.mapred.LocalJobRunnerMetrics rewritten to use new 
> metrics2 API.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to