Github user zsxwing commented on the pull request:

    https://github.com/apache/spark/pull/5130#issuecomment-84869736
  
    > Also, how were we ending up with a success before? If anything forced us 
to break out of that try block, it seems like we wouldn't call finish with 
SUCCESS . Or does YARN just assume success in the case where we shut down 
without a report? (I can look this up if you don't know).
    
    After #4773, it should not end up with a success for OutOfMemoryError.
    
    However, in my experience, if `AMRMClient.unregisterApplicationMaster` has 
not been called, Yarn will restart the AM until exceeding the max attempts. So 
if the user does not create a `SparkContext` and the driver code exits 
normally, Yarn still will restart the AM. We have an ugly fix that forcing AM 
to register and unregister with Yarn if finding AM does not register when 
exiting. 
    



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