relaxLocality in ContainerRequest

[Maysam Yabandeh]

Hi,

According to the java docs, this is what relaxLocality supposed to mean:
/*
 * @param relaxLocality
 *          If true, containers for this request may be assigned on hosts
 *          and racks other than the ones explicitly requested.
 */

However, the way it is used in AMRMClientImpl.java implies differently:
{code:java}
      for (String node : dedupedNodes) {
        addResourceRequest(req.getPriority(), node, req.getCapability(),
req,
            true);
      }
{code}
where, relaxLocality for nodes is always set to true, ignoring the original
relaxLocality set by the user. Since this is introduced from the very
beginning at YARN-521, I do not whether it is a bug or the title
relaxLocality is confusing.

Thanks
Maysam