[jira] [Commented] (MAPREDUCE-4844) Counters.java doesn't obey Java Memory Model

["Brahma Reddy Battula (JIRA)" <ji...@apache.org>]

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

Brahma Reddy Battula commented on MAPREDUCE-4844:
-------------------------------------------------

[~jira.shegalov] Kindly review the attached patch!!!

> Counters.java doesn't obey Java Memory Model
> --------------------------------------------
>
>                 Key: MAPREDUCE-4844
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4844
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>    Affects Versions: 3.0.0, 2.6.0
>            Reporter: Gera Shegalov
>            Assignee: Brahma Reddy Battula
>         Attachments: MAPREDUCE-4844-002.patch, MAPREDUCE-4844-branch-1.patch
>
>
> Counters have a number of immutable fields that have not been declared 'final'.
> For example, the field groups is not final. It is, however, accessed in a couple of methods that are declared 'synchronized'. While there is a happens-before relationship between these methods calls, there is none between the Counters object initialization and these synchronized methods.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)