pgaref commented on a change in pull request #103:
URL: https://github.com/apache/tez/pull/103#discussion_r570502957



##########
File path: tez-dag/src/main/java/org/apache/tez/client/LocalClient.java
##########
@@ -403,7 +405,14 @@ protected DAGAppMaster 
createDAGAppMaster(ApplicationAttemptId applicationAttemp
   @Override
   public TezAppMasterStatus getAMStatus(Configuration configuration, 
ApplicationId appId,
       UserGroupInformation ugi) throws TezException, ServiceException, 
IOException {
-    return clientHandler.getTezAppMasterStatus();
+    if (isLocalWithoutNetwork) {
+      if (clientHandler == null) {
+        return TezAppMasterStatus.INITIALIZING;
+      }
+      return clientHandler.getTezAppMasterStatus();
+    } else {

Review comment:
       Oh I see, this part of the TEZ-4236 addition -- makes sense




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to