)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18573,"name":"Petri Kemppainen","email":"petri.kemppainen@nokia.com","username":"petrkemp"},"change_message_id":"72dcd1abbeffb8046853983d0e4400fd7ceb2054","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently in method autonested_transaction from db.api is decided"},{"line_number":10,"context_line":" begin(nested) or begin(subtransactions) should be used based on"},{"line_number":11,"context_line":"catching the the exception. Instead of this it can be checking is"},{"line_number":12,"context_line":"there active session or not."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I2b5951bcffef99323e3d68034ae9a3fd9fa1970c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a5de9d1_5f12e263","line":11,"range":{"start_line":11,"start_character":9,"end_line":11,"end_character":16},"updated":"2016-01-28 09:33:18.000000000","message":"there seems to be 2x the","commit_id":"e072d06ce649765b9c7f8be13af09cea0ef60a0b"}],"neutron/db/api.py":[{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"7ed74cd30f522e7de4ee87966603bacafd14486b","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    session_type \u003d \u0027subtransactions\u0027"},{"line_number":84,"context_line":"    if sess.is_active:"},{"line_number":85,"context_line":"        session_type \u003d \u0027nested\u0027"},{"line_number":86,"context_line":"    with sess.begin(**{session_type: True}) as tx:"},{"line_number":87,"context_line":"        yield tx"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_b9f47d87","line":86,"range":{"start_line":86,"start_character":20,"end_line":86,"end_character":42},"updated":"2016-01-26 12:26:00.000000000","message":"This strikes me as just a little bit too tricky.  Do we need to do this?  What if we just did this:\n\n  if sess.is_active:\n      session_context \u003d sess.begin_nested():\n  else:\n      session_context \u003d sess.begin(subtransactions\u003dTrue)\n  with session_context as tx:\n      yield tx","commit_id":"69daa1d1ed683d08815fbe5b0630af2c4e47e500"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"1ca91fa99e11b73a6a6e0f64bfd35b51ba4fc291","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    session_type \u003d \u0027subtransactions\u0027"},{"line_number":84,"context_line":"    if sess.is_active:"},{"line_number":85,"context_line":"        session_type \u003d \u0027nested\u0027"},{"line_number":86,"context_line":"    with sess.begin(**{session_type: True}) as tx:"},{"line_number":87,"context_line":"        yield tx"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_f740ac37","line":86,"range":{"start_line":86,"start_character":20,"end_line":86,"end_character":42},"in_reply_to":"9a68dd71_0878d1f7","updated":"2016-01-26 17:10:44.000000000","message":"Carl proposal seems a bit more readable but both are fine for me","commit_id":"69daa1d1ed683d08815fbe5b0630af2c4e47e500"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"d3342e9f38baacd0d298e795a32360bbe2f07c03","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    session_type \u003d \u0027subtransactions\u0027"},{"line_number":84,"context_line":"    if sess.is_active:"},{"line_number":85,"context_line":"        session_type \u003d \u0027nested\u0027"},{"line_number":86,"context_line":"    with sess.begin(**{session_type: True}) as tx:"},{"line_number":87,"context_line":"        yield tx"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_0878d1f7","line":86,"range":{"start_line":86,"start_character":20,"end_line":86,"end_character":42},"in_reply_to":"9a68dd71_b9f47d87","updated":"2016-01-26 13:50:10.000000000","message":"The code you present is the same as mine. I can update, but I don\u0027t see anything tricky in mine :)","commit_id":"69daa1d1ed683d08815fbe5b0630af2c4e47e500"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"2f909375f4099b802796b3497c5a9103d160251c","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    session_type \u003d \u0027subtransactions\u0027"},{"line_number":84,"context_line":"    if sess.is_active:"},{"line_number":85,"context_line":"        session_type \u003d \u0027nested\u0027"},{"line_number":86,"context_line":"    with sess.begin(**{session_type: True}) as tx:"},{"line_number":87,"context_line":"        yield tx"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_741daabc","line":86,"range":{"start_line":86,"start_character":20,"end_line":86,"end_character":42},"in_reply_to":"9a68dd71_f740ac37","updated":"2016-01-27 14:40:41.000000000","message":"I was hoping this would be the quick one. I will update to make this more readable.","commit_id":"69daa1d1ed683d08815fbe5b0630af2c4e47e500"}]}