hadoop-mapreduce-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tom White (JIRA)" <j...@apache.org>
Subject [jira] Commented: (MAPREDUCE-2337) Remove dependence of public MapReduce API on classes in server package
Date Thu, 24 Feb 2011 02:24:38 GMT

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

Tom White commented on MAPREDUCE-2337:
--------------------------------------

bq. It doesn't seem right that ClientProtocol exposes the server-internal state type to me.

Good point. However I think that ClientProtocol (which is Private) should return a mapreduce.Cluster.JobTrackerStatus,
just like its getClusterMetrics() method returns a mapreduce.ClusterMetrics object. I'll create
a new patch.

> Remove dependence of public MapReduce API on classes in server package
> ----------------------------------------------------------------------
>
>                 Key: MAPREDUCE-2337
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-2337
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>            Reporter: Tom White
>            Assignee: Tom White
>         Attachments: MAPREDUCE-2337.patch, MAPREDUCE-2337.patch
>
>
> Cluster#getJobTrackerState() returns a org.apache.hadoop.mapreduce.server.jobtracker.State
enum, which makes the API in o.a.h.mapreduce have a dependency on the server package. It would
be better to make the public API self-contained by using an equivalent enum in the Cluster
class.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message