{"action":"created","issue":{"url":"https://api.github.com/repos/suchow/Wallace/issues/246","labels_url":"https://api.github.com/repos/suchow/Wallace/issues/246/labels{/name}","comments_url":"https://api.github.com/repos/suchow/Wallace/issues/246/comments","events_url":"https://api.github.com/repos/suchow/Wallace/issues/246/events","html_url":"https://github.com/suchow/Wallace/pull/246","id":113723518,"number":246,"title":"Handle Rollback and Close of SQLAlchemy Sessions","user":{"login":"alecpm","id":202854,"avatar_url":"https://avatars.githubusercontent.com/u/202854?v=3","gravatar_id":"","url":"https://api.github.com/users/alecpm","html_url":"https://github.com/alecpm","followers_url":"https://api.github.com/users/alecpm/followers","following_url":"https://api.github.com/users/alecpm/following{/other_user}","gists_url":"https://api.github.com/users/alecpm/gists{/gist_id}","starred_url":"https://api.github.com/users/alecpm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alecpm/subscriptions","organizations_url":"https://api.github.com/users/alecpm/orgs","repos_url":"https://api.github.com/users/alecpm/repos","events_url":"https://api.github.com/users/alecpm/events{/privacy}","received_events_url":"https://api.github.com/users/alecpm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-10-28T01:25:04Z","updated_at":"2015-10-28T23:04:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/suchow/Wallace/pulls/246","html_url":"https://github.com/suchow/Wallace/pull/246","diff_url":"https://github.com/suchow/Wallace/pull/246.diff","patch_url":"https://github.com/suchow/Wallace/pull/246.patch"},"body":"Adds a handler to rollback and close the session at the end of each request.  Simplifies session passing code.  This should address issue #224"},"comment":{"url":"https://api.github.com/repos/suchow/Wallace/issues/comments/152023938","html_url":"https://github.com/suchow/Wallace/pull/246#issuecomment-152023938","issue_url":"https://api.github.com/repos/suchow/Wallace/issues/246","id":152023938,"user":{"login":"thomasmorgan","id":8507889,"avatar_url":"https://avatars.githubusercontent.com/u/8507889?v=3","gravatar_id":"","url":"https://api.github.com/users/thomasmorgan","html_url":"https://github.com/thomasmorgan","followers_url":"https://api.github.com/users/thomasmorgan/followers","following_url":"https://api.github.com/users/thomasmorgan/following{/other_user}","gists_url":"https://api.github.com/users/thomasmorgan/gists{/gist_id}","starred_url":"https://api.github.com/users/thomasmorgan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomasmorgan/subscriptions","organizations_url":"https://api.github.com/users/thomasmorgan/orgs","repos_url":"https://api.github.com/users/thomasmorgan/repos","events_url":"https://api.github.com/users/thomasmorgan/events{/privacy}","received_events_url":"https://api.github.com/users/thomasmorgan/received_events","type":"User","site_admin":false},"created_at":"2015-10-28T23:04:47Z","updated_at":"2015-10-28T23:04:47Z","body":"OK, I've done some testing of the branch and it all runs fine. I am happy to merge it. Cris, you were going to do a review, right? Once you are happy we should merge.\r\n\r\nWhen we spoke we discussed trying to ascertain whether this will stop the issues where some notifications end up not being processed. However, I'm pretty confident the only way to test this properly is to run a full study so that test will have to wait until the next experiment is ready."}}