[ 
https://issues.apache.org/jira/browse/FLUME-1673?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13488589#comment-13488589
 ] 

Hudson commented on FLUME-1673:
-------------------------------

Integrated in flume-trunk #321 (See 
[https://builds.apache.org/job/flume-trunk/321/])
    FLUME-1673. MonitoredCounterGroup publishes this reference to platform 
MBean server in constructor. (Revision ab8dd8d570e4181e12de490bc26937f35349ff62)

     Result = SUCCESS
mpercy : 
http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=ab8dd8d570e4181e12de490bc26937f35349ff62
Files : 
* 
flume-ng-core/src/test/java/org/apache/flume/instrumentation/TestMonitoredCounterGroup.java
* 
flume-ng-core/src/main/java/org/apache/flume/instrumentation/MonitoredCounterGroup.java

                
> MonitoredCounterGroup publishes this reference to platform MBean server in 
> constructor
> --------------------------------------------------------------------------------------
>
>                 Key: FLUME-1673
>                 URL: https://issues.apache.org/jira/browse/FLUME-1673
>             Project: Flume
>          Issue Type: Bug
>    Affects Versions: v1.2.0
>            Reporter: Mike Percy
>            Assignee: Hari Shreedharan
>             Fix For: v1.3.0
>
>         Attachments: FLUME-1673-2.patch, FLUME-1673.patch
>
>
> MonitoredCounterGroup should not publish a reference to "this" in the 
> constructor. There should be a specific method call to do that explicitly.
> An article about why this is problematic: 
> http://www.ibm.com/developerworks/java/library/j-jtp0618/index.html#2

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to