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

Philip Zeyliger commented on MAPREDUCE-990:
-------------------------------------------

bq. As javadoc says JobContext is a readonly view of the job provided to tasks. 
get*TimeStamps cannot be package private methods. Framework would call them 
from a different packge. 

The framework doesn't currently call them at all.  The framework probably has 
more than the JobContext, so they ought to move somewhere else.  I don't think 
they should be part of the user API--can I delete them entirely until they 
actually get used?

-- Philip

> Making distributed cache getters in JobContext never return null
> ----------------------------------------------------------------
>
>                 Key: MAPREDUCE-990
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-990
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>            Reporter: Philip Zeyliger
>            Assignee: Philip Zeyliger
>            Priority: Minor
>         Attachments: MAPREDUCE-990.patch.txt
>
>
> MAPREDUCE-898 moved distributed cache setters and getters into Job and 
> JobContext.  Since the API is new, I'd like to propose that those getters 
> never return null, but instead always return an array, even if it's empty.
> If people don't like this change, I can instead merely update the javadoc to 
> reflect the fact that null may be returned.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to