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

Philip Zeyliger commented on MAPREDUCE-898:
-------------------------------------------

Amareshwari,

Sorry I slept through the first pass of this.  A few questions about the new 
API:

* Do most of the getters that return arrays return null or an empty array if 
there's nothing there?  This would be good to document in JavaDoc, since it's 
always confusing. 

* Are the get*Timestamps functions public because they're used within the 
framework?  It might be great for them to return int[] instead of String, and 
at least document what a "timestamp" indicates?

-- Philip

> Change DistributedCache to use new api.
> ---------------------------------------
>
>                 Key: MAPREDUCE-898
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-898
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>            Reporter: Amareshwari Sriramadasu
>            Assignee: Amareshwari Sriramadasu
>             Fix For: 0.21.0
>
>         Attachments: patch-898-1.txt, patch-898-2.txt, patch-898-3.txt, 
> patch-898-4.txt, patch-898.txt
>
>


-- 
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