{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/8007","labels_url":"https://api.github.com/repos/apache/spark/issues/8007/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/8007/comments","events_url":"https://api.github.com/repos/apache/spark/issues/8007/events","html_url":"https://github.com/apache/spark/pull/8007","id":99534176,"number":8007,"title":"[SPARK-8167] Make tasks that fail from YARN preemption not fail job","user":{"login":"mccheah","id":8975385,"avatar_url":"https://avatars.githubusercontent.com/u/8975385?v=3","gravatar_id":"","url":"https://api.github.com/users/mccheah","html_url":"https://github.com/mccheah","followers_url":"https://api.github.com/users/mccheah/followers","following_url":"https://api.github.com/users/mccheah/following{/other_user}","gists_url":"https://api.github.com/users/mccheah/gists{/gist_id}","starred_url":"https://api.github.com/users/mccheah/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mccheah/subscriptions","organizations_url":"https://api.github.com/users/mccheah/orgs","repos_url":"https://api.github.com/users/mccheah/repos","events_url":"https://api.github.com/users/mccheah/events{/privacy}","received_events_url":"https://api.github.com/users/mccheah/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":53,"created_at":"2015-08-06T22:03:35Z","updated_at":"2015-09-03T01:32:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/8007","html_url":"https://github.com/apache/spark/pull/8007","diff_url":"https://github.com/apache/spark/pull/8007.diff","patch_url":"https://github.com/apache/spark/pull/8007.patch"},"body":"The architecture is that, in YARN mode, if the driver detects that an executor has disconnected, it asks the ApplicationMaster why the executor died. If the ApplicationMaster is aware that the executor died because of preemption, all tasks associated with that executor are not marked as failed. The executor\r\nis still removed from the driver's list of available executors, however.\r\n\r\nThere's a few open questions:\r\n1. Should standalone mode have a similar \"get executor loss reason\" as well? I localized this change as much as possible to affect only YARN, but there could be a valid case to differentiate executor losses in standalone mode as well.\r\n2. I make a pretty strong assumption in YarnAllocator that getExecutorLossReason(executorId) will only be called once per executor id; I do this so that I can remove the metadata from the in-memory map to avoid object accumulation. It's not clear if I'm being overly zealous to save space, however.\r\n\r\ncc @vanzin specifically for review because it collided with some earlier YARN scheduling work.\r\ncc @JoshRosen because it's similar to output commit coordination we did in the past\r\ncc @andrewor14 for our discussion on how to get executor exit codes and loss reasons"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/137296635","html_url":"https://github.com/apache/spark/pull/8007#issuecomment-137296635","issue_url":"https://api.github.com/repos/apache/spark/issues/8007","id":137296635,"user":{"login":"AmplabJenkins","id":4031291,"avatar_url":"https://avatars.githubusercontent.com/u/4031291?v=3","gravatar_id":"","url":"https://api.github.com/users/AmplabJenkins","html_url":"https://github.com/AmplabJenkins","followers_url":"https://api.github.com/users/AmplabJenkins/followers","following_url":"https://api.github.com/users/AmplabJenkins/following{/other_user}","gists_url":"https://api.github.com/users/AmplabJenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/AmplabJenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AmplabJenkins/subscriptions","organizations_url":"https://api.github.com/users/AmplabJenkins/orgs","repos_url":"https://api.github.com/users/AmplabJenkins/repos","events_url":"https://api.github.com/users/AmplabJenkins/events{/privacy}","received_events_url":"https://api.github.com/users/AmplabJenkins/received_events","type":"User","site_admin":false},"created_at":"2015-09-03T01:32:25Z","updated_at":"2015-09-03T01:32:25Z","body":"Merged build started."}}