diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ResourceManager.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ResourceManager.java index 8f2c121e6e..f727f55168 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ResourceManager.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/ResourceManager.java @@ -59,6 +59,7 @@ import org.apache.hadoop.yarn.event.Dispatcher; import org.apache.hadoop.yarn.event.EventDispatcher; import org.apache.hadoop.yarn.event.EventHandler; +import org.apache.hadoop.yarn.exceptions.YarnException; import org.apache.hadoop.yarn.exceptions.YarnRuntimeException; import org.apache.hadoop.yarn.server.resourcemanager.ahs.RMApplicationHistoryWriter; import org.apache.hadoop.yarn.server.resourcemanager.amlauncher.AMLauncherEventType; @@ -238,13 +239,7 @@ protected void serviceInit(Configuration conf) throws Exception { rmContext.setConfigurationProvider(configurationProvider); // load core-site.xml - InputStream coreSiteXMLInputStream = - this.configurationProvider.getConfigurationInputStream(this.conf, - YarnConfiguration.CORE_SITE_CONFIGURATION_FILE); - if (coreSiteXMLInputStream != null) { - this.conf.addResource(coreSiteXMLInputStream, - YarnConfiguration.CORE_SITE_CONFIGURATION_FILE); - } + loadConfigurationXml(YarnConfiguration.CORE_SITE_CONFIGURATION_FILE); // Do refreshUserToGroupsMappings with loaded core-site.xml Groups.getUserToGroupsMappingServiceWithLoadedConfiguration(this.conf) @@ -257,13 +252,7 @@ protected void serviceInit(Configuration conf) throws Exception { ProxyUsers.refreshSuperUserGroupsConfiguration(this.conf); // load yarn-site.xml - InputStream yarnSiteXMLInputStream = - this.configurationProvider.getConfigurationInputStream(this.conf, - YarnConfiguration.YARN_SITE_CONFIGURATION_FILE); - if (yarnSiteXMLInputStream != null) { - this.conf.addResource(yarnSiteXMLInputStream, - YarnConfiguration.YARN_SITE_CONFIGURATION_FILE); - } + loadConfigurationXml(YarnConfiguration.YARN_SITE_CONFIGURATION_FILE); validateConfigs(this.conf); @@ -339,6 +328,16 @@ protected void serviceInit(Configuration conf) throws Exception { super.serviceInit(this.conf); } + private void loadConfigurationXml(String configurationFile) + throws YarnException, IOException { + InputStream configurationInputStream = + this.configurationProvider.getConfigurationInputStream(this.conf, + configurationFile); + if (configurationInputStream != null) { + this.conf.addResource(configurationInputStream, configurationFile); + } + } + protected EmbeddedElector createEmbeddedElector() throws IOException { EmbeddedElector elector; curatorEnabled =