You are viewing a plain text version of this content. The canonical link for it is here.

Posted to mapreduce-issues@hadoop.apache.org by "Todd Lipcon (JIRA)" <ji...@apache.org> on 2011/02/23 23:38:38 UTC

[jira] Commented: (MAPREDUCE-2337) Remove dependence of public MapReduce API on classes in server package

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

Todd Lipcon commented on MAPREDUCE-2337:
----------------------------------------

I think this makes sense, but maybe it actually should get pushed into the protocol package? ie the new State enum should be part of ClientProtocol, and ClientProtocol.getJobTrackerStatus() would return the new type while deprecating ClientProtocol.getJobTrackerState? It doesn't seem right that ClientProtocol exposes the server-internal state type to me.

> 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