)]}'
{"mistral/tests/unit/engine/test_error_handling.py":[{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"a2c31ac3b5e5f703d8fceda64f30a83922baf929","unresolved":false,"context_lines":[{"line_number":381,"context_line":""},{"line_number":382,"context_line":"        self.await_workflow_error(wf_ex.id)"},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"        with db_api.transaction():"},{"line_number":385,"context_line":"            wf_ex \u003d db_api.get_workflow_execution(wf_ex.id)"},{"line_number":386,"context_line":""},{"line_number":387,"context_line":"            task_ex \u003d wf_ex.task_executions[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_a777dbd2","line":384,"updated":"2016-10-20 02:05:03.000000000","message":"just curious, what\u0027s the reason to use a db transaction here?","commit_id":"91068b5f08592922b6251e684ed574c98add7a38"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"ae8f7c3f48fc6f71b93a702f3cb54fcee375c41f","unresolved":false,"context_lines":[{"line_number":381,"context_line":""},{"line_number":382,"context_line":"        self.await_workflow_error(wf_ex.id)"},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"        with db_api.transaction():"},{"line_number":385,"context_line":"            wf_ex \u003d db_api.get_workflow_execution(wf_ex.id)"},{"line_number":386,"context_line":""},{"line_number":387,"context_line":"            task_ex \u003d wf_ex.task_executions[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_787f2466","line":384,"in_reply_to":"fa6399be_184d4872","updated":"2016-10-20 04:18:11.000000000","message":"Yes, exactly. But for a different field. The thing is that it even sometimes fails on py34 and py27, but I haven\u0027t yet investigated why only sometimes. It\u0027s a little weird. Will do when I have time.","commit_id":"91068b5f08592922b6251e684ed574c98add7a38"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"e5a46442246cef9c36720b463d8223527b48814a","unresolved":false,"context_lines":[{"line_number":381,"context_line":""},{"line_number":382,"context_line":"        self.await_workflow_error(wf_ex.id)"},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"        with db_api.transaction():"},{"line_number":385,"context_line":"            wf_ex \u003d db_api.get_workflow_execution(wf_ex.id)"},{"line_number":386,"context_line":""},{"line_number":387,"context_line":"            task_ex \u003d wf_ex.task_executions[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_184d4872","line":384,"in_reply_to":"fa6399be_7d471677","updated":"2016-10-20 04:12:10.000000000","message":"Thanks for the answer. Is it the reason that the patch https://review.openstack.org/388590 failed for py35?","commit_id":"91068b5f08592922b6251e684ed574c98add7a38"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"a232e94b06c1762e2920af2c41cc5dfb1b427beb","unresolved":false,"context_lines":[{"line_number":381,"context_line":""},{"line_number":382,"context_line":"        self.await_workflow_error(wf_ex.id)"},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"        with db_api.transaction():"},{"line_number":385,"context_line":"            wf_ex \u003d db_api.get_workflow_execution(wf_ex.id)"},{"line_number":386,"context_line":""},{"line_number":387,"context_line":"            task_ex \u003d wf_ex.task_executions[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_7d471677","line":384,"in_reply_to":"fa6399be_a777dbd2","updated":"2016-10-20 04:00:37.000000000","message":"Because wf_ex.task_executions leads a lazy initialization and hence requires a DB query which must be done in an ORM session (which, in fact, has the same scope as DB API transaction).","commit_id":"91068b5f08592922b6251e684ed574c98add7a38"}]}