YARN-5297. Avoid printing a stack trace when recovering an app after the RM restarts. (Junping Du via Varun Saxena).
This commit is contained in:
parent
30ee57ceb1
commit
5252562edf
|
@ -151,7 +151,10 @@ public class ApplicationMasterService extends AbstractService implements
|
||||||
SaslRpcServer.AuthMethod.TOKEN.toString());
|
SaslRpcServer.AuthMethod.TOKEN.toString());
|
||||||
this.server = getServer(rpc, serverConf, masterServiceAddress,
|
this.server = getServer(rpc, serverConf, masterServiceAddress,
|
||||||
this.rmContext.getAMRMTokenSecretManager());
|
this.rmContext.getAMRMTokenSecretManager());
|
||||||
|
// TODO more exceptions could be added later.
|
||||||
|
this.server.addTerseExceptions(
|
||||||
|
ApplicationMasterNotRegisteredException.class);
|
||||||
|
|
||||||
// Enable service authorization?
|
// Enable service authorization?
|
||||||
if (conf.getBoolean(
|
if (conf.getBoolean(
|
||||||
CommonConfigurationKeysPublic.HADOOP_SECURITY_AUTHORIZATION,
|
CommonConfigurationKeysPublic.HADOOP_SECURITY_AUTHORIZATION,
|
||||||
|
@ -411,11 +414,6 @@ public class ApplicationMasterService extends AbstractService implements
|
||||||
String message =
|
String message =
|
||||||
"AM is not registered for known application attempt: " + appAttemptId
|
"AM is not registered for known application attempt: " + appAttemptId
|
||||||
+ " or RM had restarted after AM registered . AM should re-register.";
|
+ " or RM had restarted after AM registered . AM should re-register.";
|
||||||
LOG.info(message);
|
|
||||||
RMAuditLogger.logFailure(
|
|
||||||
this.rmContext.getRMApps().get(appAttemptId.getApplicationId())
|
|
||||||
.getUser(), AuditConstants.AM_ALLOCATE, "",
|
|
||||||
"ApplicationMasterService", message, applicationId, appAttemptId);
|
|
||||||
throw new ApplicationMasterNotRegisteredException(message);
|
throw new ApplicationMasterNotRegisteredException(message);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue