diff --git a/hadoop-yarn-project/CHANGES.txt b/hadoop-yarn-project/CHANGES.txt index addb9bec25..44fefb21ac 100644 --- a/hadoop-yarn-project/CHANGES.txt +++ b/hadoop-yarn-project/CHANGES.txt @@ -203,6 +203,8 @@ Release 2.4.0 - UNRELEASED YARN-1482. Modified WebApplicationProxy to make it work across ResourceManager fail-over. (Xuan Gong via vinodkv) + YARN-1568. Rename clusterid to clusterId in ActiveRMInfoProto (kasha) + OPTIMIZATIONS BUG FIXES diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/proto/server/yarn_server_resourcemanager_service_protos.proto b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/proto/server/yarn_server_resourcemanager_service_protos.proto index 91c80db0c8..e27f12a836 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/proto/server/yarn_server_resourcemanager_service_protos.proto +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/proto/server/yarn_server_resourcemanager_service_protos.proto @@ -138,6 +138,6 @@ message RMStateVersionProto { ///////////// RM Failover related records //////////////////////// ////////////////////////////////////////////////////////////////// message ActiveRMInfoProto { - required string clusterid = 1; + required string clusterId = 1; required string rmId = 2; } diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/EmbeddedElectorService.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/EmbeddedElectorService.java index 7b164418d3..0c09c2753c 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/EmbeddedElectorService.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/EmbeddedElectorService.java @@ -172,7 +172,7 @@ private static byte[] createActiveNodeInfo(String clusterId, String rmId) throws IOException { return YarnServerResourceManagerServiceProtos.ActiveRMInfoProto .newBuilder() - .setClusterid(clusterId) + .setClusterId(clusterId) .setRmId(rmId) .build() .toByteArray(); @@ -198,10 +198,10 @@ private synchronized boolean isParentZnodeSafe(String clusterId) } // Check if the passed proto corresponds to an RM in the same cluster - if (!proto.getClusterid().equals(clusterId)) { + if (!proto.getClusterId().equals(clusterId)) { LOG.error("Mismatched cluster! The other RM seems " + "to be from a different cluster. Current cluster = " + clusterId + - "Other RM's cluster = " + proto.getClusterid()); + "Other RM's cluster = " + proto.getClusterId()); return false; } return true;