Github user lianhuiwang commented on the pull request:

    https://github.com/apache/spark/pull/3962#issuecomment-71574630
  
    @andrewor14 if we don't finish with SUCCEEDED on driver disassociation, AM 
should finish with non-zero. example: if driver's main class throw some 
exception and exit, I think now AMActor will listen on the disassociated event 
and finish with SUCCEEDED.if AMActor donot listen on the disassociated event, 
userThread in AM can catch these exception from driver's main class and finish 
with FAILED, not SUCCEEDED.
    so i think AMActor is unnecessary to listening to driver's disassociated 
event in yarn cluster mode because userThread in AM can monitor any exceptions 
of driver's main class. but  AMActor donot know any exceptions of driver. Do 
you understan what i said? or Do you have different opinions?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to