)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"094471d5f8f274e22805f302d2110efe60d2212d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Valeriy Ponomaryov \u003cvponomaryov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-12 00:19:51 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Close migration sessions explicitly for compatibility with PyMySQL"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oslo.db lib starting with version 1.12.0 uses PyMySQL as mysql client."},{"line_number":10,"context_line":"And if we do some actions with migration sessions and do not close it, then"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a41bdd9_a4ef71af","line":7,"updated":"2015-07-12 17:03:42.000000000","message":"Please reword to get summary line within 50 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"64a9a28f3f2c8c9cbf1b3cd10eafa2d9ab7b464b"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"ddb7ad1c7c2658a8dca9c8f209570a188c00cbe6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Valeriy Ponomaryov \u003cvponomaryov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-12 00:19:51 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Close migration sessions explicitly for compatibility with PyMySQL"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oslo.db lib starting with version 1.12.0 uses PyMySQL as mysql client."},{"line_number":10,"context_line":"And if we do some actions with migration sessions and do not close it, then"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a41bdd9_933143a5","line":7,"in_reply_to":"9a41bdd9_a4ef71af","updated":"2015-07-13 06:08:57.000000000","message":"Length up to 72 symbols is allowed. I leave it as is.","commit_id":"64a9a28f3f2c8c9cbf1b3cd10eafa2d9ab7b464b"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"094471d5f8f274e22805f302d2110efe60d2212d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oslo.db lib starting with version 1.12.0 uses PyMySQL as mysql client."},{"line_number":10,"context_line":"And if we do some actions with migration sessions and do not close it, then"},{"line_number":11,"context_line":"alembic, as db migration tool does not write to DB updated migration version."},{"line_number":12,"context_line":"It breaks further attempts to perform some migration actions in case some"},{"line_number":13,"context_line":"migration can not be applied twice."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a41bdd9_a41d31bf","line":11,"updated":"2015-07-12 17:03:42.000000000","message":"This sentence is confusing. Could you reword it to better state what the issue is when sessions are not closed?","commit_id":"64a9a28f3f2c8c9cbf1b3cd10eafa2d9ab7b464b"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"ddb7ad1c7c2658a8dca9c8f209570a188c00cbe6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oslo.db lib starting with version 1.12.0 uses PyMySQL as mysql client."},{"line_number":10,"context_line":"And if we do some actions with migration sessions and do not close it, then"},{"line_number":11,"context_line":"alembic, as db migration tool does not write to DB updated migration version."},{"line_number":12,"context_line":"It breaks further attempts to perform some migration actions in case some"},{"line_number":13,"context_line":"migration can not be applied twice."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a41bdd9_33ab978b","line":11,"in_reply_to":"9a41bdd9_a41d31bf","updated":"2015-07-13 06:08:57.000000000","message":"I know that not closed session is the root cause for problem when used PyMySQL as a client. \"How exactly\" it is broken - I do not know. So, if I understand you correctly, in that case nothing to reword here. Or, you can propose such rewording.","commit_id":"64a9a28f3f2c8c9cbf1b3cd10eafa2d9ab7b464b"}]}
