rbalamohan commented on code in PR #280:
URL: https://github.com/apache/tez/pull/280#discussion_r1162316396
##########
tez-dag/src/main/java/org/apache/tez/dag/app/DAGAppMaster.java:
##########
@@ -587,6 +588,8 @@ protected void serviceInit(final Configuration conf) throws
Exception {
taskSchedulerManager);
addIfServiceDependency(taskSchedulerManager, clientRpcServer);
+ appMasterReadinessService = createAppMasterReadinessService();
Review Comment:
Issue is happening as a corner case, as the taskscheduler could be taking
longer to init. In this case, possibly "LlapTaskSchedulerService" is taking
time to init due to ZK dependency.
Instead of adding one more dependency, why not throw RuntimeException in
TaskSchedulerManager::getTaskSchedulerClassName (i.e until all taskSchedulers
are initied w.r.t appContext) ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
