[
https://issues.apache.org/jira/browse/MAPREDUCE-744?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Devaraj Das updated MAPREDUCE-744:
----------------------------------
Attachment: 744-4.patch
bq. 1. DistributedCache.get*Visibiliteies - are deprecated. Why do we add
deprecated methods?
MAPREDUCE-982 needs to be fixed for me to not have the deprecated methods. But
i will think about if we can avoid it in any way.
bq. 2. Do we need separate call for determineFile/ArchiveVisability. It can be
determined while calling DistributedCache.addCacheArchive/File()
I don't have a strong preference for this either way but it seems clearer to me
to keep it the way i currently have..
bq. 3. private static isPublic() should be called differently - something like
haveOtherPermissions(Perm), to match its functionality. Some comments would
help too.
Done
bq. 4. public methods should have javadoc comments
(determineCacheVisibilities() in TrackerDistributedCacheManager)
Done
bq. 5. this line: File workDir = new File(new Path(TEST_ROOT_DIR,
"workdir").toString()); can be changed
Done
> Support in DistributedCache to share cache files with other users after
> HADOOP-4493
> -----------------------------------------------------------------------------------
>
> Key: MAPREDUCE-744
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-744
> Project: Hadoop Map/Reduce
> Issue Type: Sub-task
> Components: tasktracker
> Reporter: Vinod K V
> Assignee: Devaraj Das
> Fix For: 0.22.0
>
> Attachments: 744-1.patch, 744-2.patch, 744-3.patch, 744-4.patch,
> 744-early.patch
>
>
> HADOOP-4493 aims to completely privatize the files distributed to TT via
> DistributedCache. This jira issues focuses on sharing some/all of these files
> with all other users.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
