{"action":"created","issue":{"url":"https://api.github.com/repos/spotify/luigi/issues/1205","labels_url":"https://api.github.com/repos/spotify/luigi/issues/1205/labels{/name}","comments_url":"https://api.github.com/repos/spotify/luigi/issues/1205/comments","events_url":"https://api.github.com/repos/spotify/luigi/issues/1205/events","html_url":"https://github.com/spotify/luigi/pull/1205","id":105862155,"number":1205,"title":"Add an option to prevent Worker from catching exceptions from Task.run()","user":{"login":"belevtsoff","id":582489,"avatar_url":"https://avatars.githubusercontent.com/u/582489?v=3","gravatar_id":"","url":"https://api.github.com/users/belevtsoff","html_url":"https://github.com/belevtsoff","followers_url":"https://api.github.com/users/belevtsoff/followers","following_url":"https://api.github.com/users/belevtsoff/following{/other_user}","gists_url":"https://api.github.com/users/belevtsoff/gists{/gist_id}","starred_url":"https://api.github.com/users/belevtsoff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/belevtsoff/subscriptions","organizations_url":"https://api.github.com/users/belevtsoff/orgs","repos_url":"https://api.github.com/users/belevtsoff/repos","events_url":"https://api.github.com/users/belevtsoff/events{/privacy}","received_events_url":"https://api.github.com/users/belevtsoff/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-09-10T17:52:04Z","updated_at":"2015-09-10T22:27:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/spotify/luigi/pulls/1205","html_url":"https://github.com/spotify/luigi/pull/1205","diff_url":"https://github.com/spotify/luigi/pull/1205.diff","patch_url":"https://github.com/spotify/luigi/pull/1205.patch"},"body":"Addresses #1201 \r\n\r\nHere I used the fact that `multiprocessing.Queue` does basic serialization itself, which should work well for exceptions. I also wanted to create a container class TaskProcessResult to formalize a bit the growing set of attributes that is passed over the queue and read in several places, but I have no experience with mock and didn't know how to adjust the tests. Do you think that would be an overkill?"},"comment":{"url":"https://api.github.com/repos/spotify/luigi/issues/comments/139399998","html_url":"https://github.com/spotify/luigi/pull/1205#issuecomment-139399998","issue_url":"https://api.github.com/repos/spotify/luigi/issues/1205","id":139399998,"user":{"login":"Tarrasch","id":294349,"avatar_url":"https://avatars.githubusercontent.com/u/294349?v=3","gravatar_id":"","url":"https://api.github.com/users/Tarrasch","html_url":"https://github.com/Tarrasch","followers_url":"https://api.github.com/users/Tarrasch/followers","following_url":"https://api.github.com/users/Tarrasch/following{/other_user}","gists_url":"https://api.github.com/users/Tarrasch/gists{/gist_id}","starred_url":"https://api.github.com/users/Tarrasch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tarrasch/subscriptions","organizations_url":"https://api.github.com/users/Tarrasch/orgs","repos_url":"https://api.github.com/users/Tarrasch/repos","events_url":"https://api.github.com/users/Tarrasch/events{/privacy}","received_events_url":"https://api.github.com/users/Tarrasch/received_events","type":"User","site_admin":false},"created_at":"2015-09-10T22:27:53Z","updated_at":"2015-09-10T22:27:53Z","body":"@belevtsoff Yea, I think that keeps the commit log clener. And when you git blame the test lines, you'll also see the implementation that came with it, and vice verse."}}