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

Tom White commented on MAPREDUCE-4105:
--------------------------------------

Is the bug that setConf() isn't being called on DNSToSwitchMapping? The new 
test doesn't actually test that case. (The test could go in TestRackResolver - 
it doesn't need a new test class - and could pass in a mock DNSToSwitchMapping 
that checks that its setConf() is called.) Rather than adding a new method to 
the DNSToSwitchMapping interface, can you use ReflectionUtils.newInstance? This 
will take care of calling setConf() if the DNSToSwitchMapping implements 
Configurable.

                
> Yarn RackResolver ignores rack configurations
> ---------------------------------------------
>
>                 Key: MAPREDUCE-4105
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4105
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: mrv2
>    Affects Versions: 0.23.1
>            Reporter: Ahmed Radwan
>            Assignee: Ahmed Radwan
>             Fix For: 0.23.2
>
>         Attachments: MAPREDUCE-4105.patch
>
>
> Incorrect mappings because the Yarn RackResolver ignores rack configurations. 
> This can be verified by inspecting the resource manager web ui that lists all 
> the nodes, all of them show up with /default-rack regardless of the output 
> from the script specified using net.topology.script.file.name configuration 
> property.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to