Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/6409#issuecomment-105604628
As Tom says, this change might break application retries. I think it's time
to write a test so that people don't break that part of the code.
The test should set up max tries to two, force the first attempt to fail,
and make sure the second succeeds (and that two attempts were run).
For this particular change, perhaps the right choice would be to relax the
condition so that it also includes "KILLED" aside from "SUCCEEDED".
---
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]
