{"action":"created","issue":{"url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/issues/391","repository_url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator","labels_url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/issues/391/labels{/name}","comments_url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/issues/391/comments","events_url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/issues/391/events","html_url":"https://github.com/lightblue-platform/lightblue-migrator/pull/391","id":171488683,"number":391,"title":"Update timestampInitialValue in the same try-catch block as generated jobs insertion","user":{"login":"paterczm","id":4681035,"avatar_url":"https://avatars.githubusercontent.com/u/4681035?v=3","gravatar_id":"","url":"https://api.github.com/users/paterczm","html_url":"https://github.com/paterczm","followers_url":"https://api.github.com/users/paterczm/followers","following_url":"https://api.github.com/users/paterczm/following{/other_user}","gists_url":"https://api.github.com/users/paterczm/gists{/gist_id}","starred_url":"https://api.github.com/users/paterczm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paterczm/subscriptions","organizations_url":"https://api.github.com/users/paterczm/orgs","repos_url":"https://api.github.com/users/paterczm/repos","events_url":"https://api.github.com/users/paterczm/events{/privacy}","received_events_url":"https://api.github.com/users/paterczm/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/labels/bug","name":"bug","color":"e11d21"},{"url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/labels/ready%20for%20review","name":"ready for review","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-08-16T18:45:19Z","updated_at":"2016-08-16T18:47:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/pulls/391","html_url":"https://github.com/lightblue-platform/lightblue-migrator/pull/391","diff_url":"https://github.com/lightblue-platform/lightblue-migrator/pull/391.diff","patch_url":"https://github.com/lightblue-platform/lightblue-migrator/pull/391.patch"},"body":"It is possible for job insertion to fail but timestampInitialValue update to succeed. As a result, some generated jobs may be missing and data for those periods will not be checked for consistency. Those cases are not easy to catch.\r\n\r\nTo fix this, I'm putting timestampInitialValue update in the same try-catch block as generated jobs insertion. It is still possible that timestampInitialValue update fails after successful job insertion, resulting in job duplicates, but that's better than missing jobs."},"comment":{"url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/issues/comments/240199603","html_url":"https://github.com/lightblue-platform/lightblue-migrator/pull/391#issuecomment-240199603","issue_url":"https://api.github.com/repos/lightblue-platform/lightblue-migrator/issues/391","id":240199603,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars.githubusercontent.com/u/2354108?v=3","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2016-08-16T18:47:42Z","updated_at":"2016-08-16T18:47:42Z","body":"\n[![Coverage Status](https://coveralls.io/builds/7473974/badge)](https://coveralls.io/builds/7473974)\n\nCoverage decreased (-67.7%) to 0.0% when pulling **84235aa31009ac2888cac5372a4353fadac19bf9 on ccex** into **d861ccfbdf9f51f409492cc95c1e0cd66266f548 on master**.\n"}}
{"actor":{"display_login":"coveralls"}}