[ https://issues.apache.org/jira/browse/HADOOP-1985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12560675#action_12560675 ]
Hadoop QA commented on HADOOP-1985: ----------------------------------- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12373500/1985.v9.patch against trunk revision r613359. @author +1. The patch does not contain any @author tags. javadoc +1. The javadoc tool did not generate any warning messages. javac +1. The applied patch does not generate any new compiler warnings. findbugs +1. The patch does not introduce any new Findbugs warnings. core tests -1. The patch failed core unit tests. contrib tests +1. The patch passed contrib unit tests. Test results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1648/testReport/ Findbugs warnings: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1648/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1648/artifact/trunk/build/test/checkstyle-errors.html Console output: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1648/console This message is automatically generated. > Abstract node to switch mapping into a topology service class used by > namenode and jobtracker > --------------------------------------------------------------------------------------------- > > Key: HADOOP-1985 > URL: https://issues.apache.org/jira/browse/HADOOP-1985 > Project: Hadoop > Issue Type: New Feature > Components: dfs, mapred > Reporter: eric baldeschwieler > Assignee: Devaraj Das > Fix For: 0.16.0 > > Attachments: 1985.new.patch, 1985.v1.patch, 1985.v2.patch, > 1985.v3.patch, 1985.v4.patch, 1985.v5.patch, 1985.v6.patch, 1985.v9.patch, > jobinprogress.patch > > > In order to implement switch locality in MapReduce, we need to have switch > location in both the namenode and job tracker. Currently the namenode asks > the data nodes for this info and they run a local script to answer this > question. In our environment and others that I know of there is no reason to > push this to each node. It is easier to maintain a centralized script that > maps node DNS names to switch strings. > I propose that we build a new class that caches known DNS name to switch > mappings and invokes a loadable class or a configurable system call to > resolve unknown DNS to switch mappings. We can then add this to the namenode > to support the current block to switch mapping needs and simplify the data > nodes. We can also add this same callout to the job tracker and then > implement rack locality logic there without needing to chane the filesystem > API or the split planning API. > Not only is this the least intrusive path to building racklocal MR I can ID, > it is also future compatible to future infrastructures that may derive > topology on the fly, etc, etc... -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.