[ https://issues.apache.org/jira/browse/MAPREDUCE-1938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12900388#action_12900388 ]
Luke Lu commented on MAPREDUCE-1938: ------------------------------------ A couple of questions regarding the patch (filename should probably end with .patch): # Are the new property a system/cluster property or a per job property? i.e, allowing user to to override the behavior (default to system jars first) per job. Owen's comments seem to imply it's a per job switch, which should be defined/documented in MRJobConfig (MAPREDUCE-1749) # Why there are two boolean properties defined in the trunk patch "mapreduce.user.classpath.first" (defined in TaskRunner) and "mapreduce.task.classpath.user.precedence" (defined in JobContext)? The latter seems to have no effect and not tested. # Why use Vector (in the test code) when there doesn't seem to be need for the list to be synchronized? # Why do the test code use hard code paths and string literals when there are corresponding constants defined? > Ability for having user's classes take precedence over the system classes for > tasks' classpath > ---------------------------------------------------------------------------------------------- > > Key: MAPREDUCE-1938 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-1938 > Project: Hadoop Map/Reduce > Issue Type: New Feature > Components: job submission, task, tasktracker > Reporter: Devaraj Das > Fix For: 0.22.0 > > Attachments: mapred-1938-1, mr-1938-bp20.1.patch, mr-1938-bp20.patch > > > It would be nice to have the ability in MapReduce to allow users to specify > for their jobs alternate implementations of classes that are already defined > in the MapReduce libraries. For example, an alternate implementation for > CombineFileInputFormat. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.