[
https://issues.apache.org/jira/browse/HADOOP-4041?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Philip Zeyliger updated HADOOP-4041:
------------------------------------
Comment: was deleted
(was: bq. In DistributedCacheHandle the class doc should go before the class
declaration, not at the beginning of the file. Also need to add Apache license.
Done.
bq. Use an enum rather than a boolean for isArchive in CacheFile.
Done.
bq. We shouldn't remove public methods to DistributedCache, but rather
deprecate them and remove them in a future release. Can DistributedCache
delegate to DistributedCacheManager? I like the fact you have documented the
intended audience for each public method of DistributedCache. (This paves the
way to separating the public and private interfaces in future.)
Done.
My current thinking on APIs (for a future JIRA) is that users should access
DistributedCache through Job.addToCache(URI, flags) and
Context.getCachedFiles(). But there's some more work to get there.
bq. Is there duplication between TestMRWithDistributedCache and tests that use
MRCaching that could be avoided?
Probably, but it's hard to tease out. MRCaching is more complicated than the
test I'm adding, and does, I believe, test some things that I don't. On the
other hand, TestMRWithDistributedCache tests the classpath stuff. I'm loath to
delete tests too eagerly.
bq. Could TestMRWithDistributedCache also test symlinking?
It does now test symlinking. However, I couldn't (easily) get LocalJobRunner
to do symlinks appropriately. LocalJobRunner doesn't currently have a notion
of task directory, and I think this patch is already quite large.)
> IsolationRunner does not work as documented
> -------------------------------------------
>
> Key: HADOOP-4041
> URL: https://issues.apache.org/jira/browse/HADOOP-4041
> Project: Hadoop Core
> Issue Type: Bug
> Components: documentation, mapred
> Affects Versions: 0.18.0
> Reporter: Yuri Pradkin
> Assignee: Philip Zeyliger
> Attachments: HADOOP-4041-v2.patch, HADOOP-4041-v3.patch,
> HADOOP-4041-v4.patch, hadoop-4041.patch,
> org.apache.hadoop.fs.LocalDirAllocator.html
>
>
> IsolationRunner does not work as documented in the tutorial.
> The tutorial says "To use the IsolationRunner, first set
> keep.failed.tasks.files to true (also see keep.tasks.files.pattern)."
> Should be:
> keep.failed.task.files (not tasks)
> After the above was set (quoted from my message on hadoop-core):
> > After the task
> > hung, I failed it via the web interface. Then I went to the node that was
> > running this task
> >
> > $ cd ...local/taskTracker/jobcache/job_200808071645_0001/work
> > (this path is already different from the tutorial's)
> >
> > $ hadoop org.apache.hadoop.mapred.IsolationRunner ../job.xml
> > Exception in thread "main" java.lang.NullPointerException
> > at
> > org.apache.hadoop.mapred.IsolationRunner.main(IsolationRunner.java:164)
> >
> > Looking at IsolationRunner code, I see this:
> >
> > 164 File workDirName = new File(lDirAlloc.getLocalPathToRead(
> > 165 TaskTracker.getJobCacheSubdir()
> > 166 + Path.SEPARATOR +
> > taskId.getJobID()
> > 167 + Path.SEPARATOR + taskId
> > 168 + Path.SEPARATOR + "work",
> > 169 conf). toString());
> >
> > I.e. it assumes there is supposed to be a taskID subdirectory under the job
> > dir, but:
> > $ pwd
> > ...mapred/local/taskTracker/jobcache/job_200808071645_0001
> > $ ls
> > jars job.xml work
> >
> > -- it's not there.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
