diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesRequest.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesRequest.java index 94814e9053..28e4db6ee8 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesRequest.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesRequest.java @@ -56,7 +56,7 @@ public static GetAttributesToNodesRequest newInstance( * Set node attributeKeys for which the mapping of hostname to attribute value * is required. * - * @param attributes Set provided. + * @param attributes Set of NodeAttributeKey provided. */ @Public @Unstable @@ -66,7 +66,7 @@ public static GetAttributesToNodesRequest newInstance( * Get node attributeKeys for which mapping of hostname to attribute value is * required. * - * @return Set + * @return Set of NodeAttributeKey */ @Public @Unstable diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesResponse.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesResponse.java index c83785fc6a..045deac7b8 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesResponse.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetAttributesToNodesResponse.java @@ -56,8 +56,7 @@ public abstract void setAttributeToNodes( * Get mapping of NodeAttributeKey to its associated mapping of list of * NodeToAttributeValue associated with attribute. * - * @return Map> node attributes - * to list of NodeToAttributeValuenode. + * @return Map of node attributes to list of NodeToAttributeValue. */ @Public @Evolving diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesRequest.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesRequest.java index 8e91bcafed..4fcd8da693 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesRequest.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesRequest.java @@ -57,7 +57,7 @@ public static GetNodesToAttributesRequest newInstance(Set hostNames) { /** * Get hostnames for which mapping is required. * - * @return Set of hostnames. + * @return Set of hostnames. */ @InterfaceAudience.Public @InterfaceStability.Evolving diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesResponse.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesResponse.java index acc07bb184..a82a3f99be 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesResponse.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/protocolrecords/GetNodesToAttributesResponse.java @@ -55,7 +55,7 @@ public static GetNodesToAttributesResponse newInstance( /** * Get hostnames to NodeAttributes mapping. * - * @return Map> host to attributes. + * @return Map of host to attributes. */ @Public @Evolving diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/nodelabels/NodeAttributesManager.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/nodelabels/NodeAttributesManager.java index ca04e8da96..a4c90a420a 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/nodelabels/NodeAttributesManager.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/nodelabels/NodeAttributesManager.java @@ -103,7 +103,7 @@ Map> getAttributesToNodes( /** * NodeAttribute to AttributeValue Map. * - * @return Map mapping of Attribute to Value. + * @return Map of NodeAttribute to AttributeValue. */ public abstract Map getAttributesForNode( String hostName); @@ -111,7 +111,7 @@ public abstract Map getAttributesForNode( /** * Get All node to Attributes list based on filter. * - * @return List nodeToAttributes matching filter.If empty + * @return List of NodeToAttributes matching filter. If empty * or null is passed as argument will return all. */ public abstract List getNodeToAttributes( @@ -120,8 +120,8 @@ public abstract List getNodeToAttributes( /** * Get all node to Attributes mapping. * - * @return Map> nodesToAttributes matching - * filter.If empty or null is passed as argument will return all. + * @return Map of String to Set of nodesToAttributes matching + * filter. If empty or null is passed as argument will return all. */ public abstract Map> getNodesToAttributes( Set hostNames); diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/nodelabels/NodeAttributesManagerImpl.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/nodelabels/NodeAttributesManagerImpl.java index 55334365ab..e5247888d7 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/nodelabels/NodeAttributesManagerImpl.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/nodelabels/NodeAttributesManagerImpl.java @@ -279,8 +279,7 @@ private void replaceNodeToAttribute(String nodeHost, String prefix, * @param nodeAttributeMapping * @param newAttributesToBeAdded * @param isRemoveOperation : to indicate whether its a remove operation. - * @return Map>, node -> Map( - * NodeAttribute -> AttributeValue) + * @return Map of String to Map of NodeAttribute to AttributeValue * @throws IOException : on invalid mapping in the current request or against * already existing NodeAttributes. */