Github user mccheah commented on the pull request:

    https://github.com/apache/spark/pull/8007#issuecomment-130889332
  
    Yeah I noticed that removeExecutor was fired from the application master as 
well. That's okay - in this code I specifically also encode the executor loss 
reason in the removeExecutor message whether or not Application master sends it 
or the DriverEndpoint sends it (via onDisconnected). But I did have to 
specifically override DriverEndpoint to make sure it sends the same message 
that the ApplicationMaster would have.


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