{"action":"created","issue":{"url":"https://api.github.com/repos/Bitergia/mordred/issues/17","repository_url":"https://api.github.com/repos/Bitergia/mordred","labels_url":"https://api.github.com/repos/Bitergia/mordred/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/Bitergia/mordred/issues/17/comments","events_url":"https://api.github.com/repos/Bitergia/mordred/issues/17/events","html_url":"https://github.com/Bitergia/mordred/pull/17","id":206187416,"number":17,"title":"Improve the Exception handling for threads by using queues. Fix T2031","user":{"login":"sanacl","id":326719,"avatar_url":"https://avatars.githubusercontent.com/u/326719?v=3","gravatar_id":"","url":"https://api.github.com/users/sanacl","html_url":"https://github.com/sanacl","followers_url":"https://api.github.com/users/sanacl/followers","following_url":"https://api.github.com/users/sanacl/following{/other_user}","gists_url":"https://api.github.com/users/sanacl/gists{/gist_id}","starred_url":"https://api.github.com/users/sanacl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sanacl/subscriptions","organizations_url":"https://api.github.com/users/sanacl/orgs","repos_url":"https://api.github.com/users/sanacl/repos","events_url":"https://api.github.com/users/sanacl/events{/privacy}","received_events_url":"https://api.github.com/users/sanacl/received_events","type":"User","site_admin":false},"labels":[{"id":430199742,"url":"https://api.github.com/repos/Bitergia/mordred/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-02-08T12:41:43Z","updated_at":"2017-02-23T14:19:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Bitergia/mordred/pulls/17","html_url":"https://github.com/Bitergia/mordred/pull/17","diff_url":"https://github.com/Bitergia/mordred/pull/17.diff","patch_url":"https://github.com/Bitergia/mordred/pull/17.patch"},"body":"Previous implementation was not raising correctly the exceptions, due to there was not method to control the exceptions launched from the mother/father process. \r\n\r\nHow to reproduce the error?\r\n- included this mbox data source in .json file\r\n```\r\n\t    \"mbox\":[\r\n\t    \t\"/home/luis/repos/mordred/mboxes\"\r\n\t    ]\r\n```\r\n- executed with update mode\r\n- monitored the number of threads\r\n```\r\nwatch -n1 \"ps huH p `ps aux|grep mordred|grep python|awk {'print $2'}`\"\r\n```\r\n- while the error is printed in the terminal were mordred was executed, the error is not logged and one of the threads dies\r\n\r\n\r\nCurrent changes allow threads to share exceptions using a queue, so the main process can wait and read if any exception was raised and needs to be addressed.\r\n\r\nThe main process does not wait and check always, in this case the exception is raised from the thread but the traceback is not printed, as this is something done by the parent. This is something to be improved.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/Bitergia/mordred/issues/comments/282003029","html_url":"https://github.com/Bitergia/mordred/pull/17#issuecomment-282003029","issue_url":"https://api.github.com/repos/Bitergia/mordred/issues/17","id":282003029,"user":{"login":"sanacl","id":326719,"avatar_url":"https://avatars.githubusercontent.com/u/326719?v=3","gravatar_id":"","url":"https://api.github.com/users/sanacl","html_url":"https://github.com/sanacl","followers_url":"https://api.github.com/users/sanacl/followers","following_url":"https://api.github.com/users/sanacl/following{/other_user}","gists_url":"https://api.github.com/users/sanacl/gists{/gist_id}","starred_url":"https://api.github.com/users/sanacl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sanacl/subscriptions","organizations_url":"https://api.github.com/users/sanacl/orgs","repos_url":"https://api.github.com/users/sanacl/repos","events_url":"https://api.github.com/users/sanacl/events{/privacy}","received_events_url":"https://api.github.com/users/sanacl/received_events","type":"User","site_admin":false},"created_at":"2017-02-23T14:19:45Z","updated_at":"2017-02-23T14:19:45Z","body":"I'm updating the code now so the merge with master is painless."}}
{"actor":{"display_login":"sanacl"}}