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

Alejandro Abdelnur commented on MAPREDUCE-3697:
-----------------------------------------------

@Mahadev, glad to help.

A few comments after browsing the patch:

* there is commented out code in the Counters class, it should be removed
* The name of the instance var 'realObject' seems to vague confusing, I'd 
suggest 'realGroup' (and the Counter 'counter' should be 'realCounter') 
* I assume that the Group iterator() will return Counters.Counter instances 
because that is what is in the groups, because fo the groupfactory. right? Else 
the iterator should have a wrapper that returns the Counter wrapper as elements.

PS: I find the mapreduce Counters stuff a bit too complicated (like why 
different group implementations), but that is not the scope of this JIRA.
                
> Hadoop Counters API limits Oozie's working across different hadoop versions
> ---------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-3697
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-3697
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: mrv2
>    Affects Versions: 0.23.1
>            Reporter: John George
>            Assignee: Mahadev konar
>            Priority: Blocker
>         Attachments: MAPREDUCE-3697-facade.patch, 
> MAPREDUCE-3697-facadeV2.patch, MAPREDUCE-3697-facadeV3.patch, 
> MAPREDUCE-3697.patch, MAPREDUCE-3697.patch, MAPREDUCE-3697.patch
>
>
> Oozie uses Hadoop Counters API, by invoking Counters.getGroup(). However, in
> hadoop 23, org.apache.hadoop.mapred.Counters does not implement getGroup(). 
> Its
> parent class AbstractCounters implements it. This is different from hadoop20X.
> As a result, Oozie compiled with either hadoop version does not work with the
> other version.
> A specific scenario, Oozie compiled with .23 and run against 205, does not
> update job status owing to a Counters API exception.
> Will explicit re-compilation against the relevant hadoop jars be required each
> time? This will prevent launching a uniform Oozie version across different
> clusters.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to