)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"change_message_id":"c54a0480c05bf05aea2d73e391ebe766759700fe","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug in db.instance_test_and_set()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If session is send via method argumetns it will raise Exception that"},{"line_number":10,"context_line":"subtransactions\u003dTrue is required."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"blueprint db-session-cleanup"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAM3%2F%2F5DM%3D","line":9,"updated":"2012-12-24 21:20:22.000000000","message":"If you need to go around again it would be nice to fix this typo.","commit_id":"515d6d2a4004c46bf46060b40c850c5eaf5bdb11"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"3bd0a7a792dd6e261c1a788d42a9dd4d8fa69bc1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix bug in db.instance_test_and_set()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If session is send via method arguments it will raise Exception that"},{"line_number":10,"context_line":"subtransactions\u003dTrue is required."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"blueprint db-session-cleanup"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAM3%2F%2F4XM%3D","line":9,"updated":"2012-12-27 01:40:10.000000000","message":"send\u003d\u003esent","commit_id":"03b3f623149f56cb32e77f67a4fbbfe093307595"}],"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"aea53ae1ed0d9d128a0f3a77c386262e3e72f31a","unresolved":false,"context_lines":[{"line_number":1774,"context_line":"    \"\"\""},{"line_number":1775,"context_line":"    session \u003d get_session()"},{"line_number":1776,"context_line":"    with session.begin():"},{"line_number":1777,"context_line":"        if not uuidutils.is_uuid_like(instance_uuid):"},{"line_number":1778,"context_line":"            raise exception.InvalidUUID(instance_uuid)"},{"line_number":1779,"context_line":""},{"line_number":1780,"context_line":"        query \u003d model_query(context, models.Instance, session\u003dsession,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2F3r8%3D","line":1777,"updated":"2012-12-28 18:08:47.000000000","message":"Since is_uuid_like is not being used to build a query, let\u0027s move this before the session.begin call.","commit_id":"f0f0d14f5f958fa4390259dc9bff6e98e2c75a76"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"aea53ae1ed0d9d128a0f3a77c386262e3e72f31a","unresolved":false,"context_lines":[{"line_number":1784,"context_line":"        attr_column \u003d getattr(models.Instance, attr)"},{"line_number":1785,"context_line":"        filter_op \u003d None"},{"line_number":1786,"context_line":"        # note(boris-42): `SELECT IN` doesn\u0027t work with None values"},{"line_number":1787,"context_line":"        # because they are incomparably."},{"line_number":1788,"context_line":"        if None in ok_states:"},{"line_number":1789,"context_line":"            ok_states.remove(None)"},{"line_number":1790,"context_line":"            filter_op \u003d or_(attr_column \u003d\u003d None, attr_column.in_(ok_states))"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2F3rk%3D","line":1787,"updated":"2012-12-28 18:08:47.000000000","message":"Three nits with this comment. \"# note\" should be \"# NOTE\". Second line should match indentation of first line. And there\u0027s a tiny typo at the end: s/incomparably/incomparable/.","commit_id":"f0f0d14f5f958fa4390259dc9bff6e98e2c75a76"},{"author":{"_account_id":2889,"name":"Aeva Black","email":"aeva.online@gmail.com","username":"tenbrae"},"change_message_id":"aea53ae1ed0d9d128a0f3a77c386262e3e72f31a","unresolved":false,"context_lines":[{"line_number":1786,"context_line":"        # note(boris-42): `SELECT IN` doesn\u0027t work with None values"},{"line_number":1787,"context_line":"        # because they are incomparably."},{"line_number":1788,"context_line":"        if None in ok_states:"},{"line_number":1789,"context_line":"            ok_states.remove(None)"},{"line_number":1790,"context_line":"            filter_op \u003d or_(attr_column \u003d\u003d None, attr_column.in_(ok_states))"},{"line_number":1791,"context_line":"        else:"},{"line_number":1792,"context_line":"            filter_op \u003d attr_column.in_(ok_states)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2F3rU%3D","line":1789,"updated":"2012-12-28 18:08:47.000000000","message":"I think we should probably not modify ok_states here, as this isn\u0027t obvious to anyone calling this method.","commit_id":"f0f0d14f5f958fa4390259dc9bff6e98e2c75a76"}]}