[ https://issues.apache.org/jira/browse/MAPREDUCE-2661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13062055#comment-13062055 ]
Josh Wills commented on MAPREDUCE-2661: --------------------------------------- +1 An optional thought: add a comment to the definition of the getSplitsAsString() method in TaskImpl.java to specify that it will return the empty string for reduce tasks since they don't have splits. > Accessing MapTaskImpl from TaskImpl > ----------------------------------- > > Key: MAPREDUCE-2661 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-2661 > Project: Hadoop Map/Reduce > Issue Type: Bug > Components: mrv2 > Reporter: Ahmed Radwan > Assignee: Ahmed Radwan > Priority: Minor > Attachments: MAPREDUCE-2661.patch > > > We are directly accessing MapTaskImpl in > TaskImpl.InitialScheduleTransition.transition(..). It'll be better to > reorganize the code so each subclass can provide its own behavior instead of > explicitly checking for the subclass type. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira