)]}'
{"keystone/revoke/backends/sql.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"68838e97b61cd24f0f50cdc38b4c6ae45ad9103c","unresolved":false,"context_lines":[{"line_number":43,"context_line":"        oldest \u003d base.revoked_before_cutoff_time()"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        with sql.session_for_write() as session:"},{"line_number":46,"context_line":"            query \u003d session.query(RevocationEvent.id)"},{"line_number":47,"context_line":"            query \u003d query.filter(RevocationEvent.revoked_at \u003c oldest)"},{"line_number":48,"context_line":"            delete_query \u003d (session.query(RevocationEvent)."},{"line_number":49,"context_line":"                            filter(RevocationEvent.id.in_(query)))"},{"line_number":50,"context_line":"            while True:"},{"line_number":51,"context_line":"                rowcount \u003d delete_query.delete(synchronize_session\u003dFalse)"},{"line_number":52,"context_line":"                if rowcount \u003d\u003d 0:"},{"line_number":53,"context_line":"                    break"},{"line_number":54,"context_line":"            else:"},{"line_number":55,"context_line":"                query \u003d session.query(RevocationEvent)"},{"line_number":56,"context_line":"                query \u003d query.filter(RevocationEvent.revoked_at \u003c oldest)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad45d7e_54b2e406","line":53,"range":{"start_line":46,"start_character":12,"end_line":53,"end_character":25},"updated":"2016-08-11 06:45:58.000000000","message":"looks like this code was never really run in tempest cause of the default batch size","commit_id":"10cc460e2a41ffd6fa713c3b7cd1052b8ef41b6e"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"55a0f6852fe7c21f84c6ed427acbbfd1239d73a1","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                rowcount \u003d delete_query.delete(synchronize_session\u003dFalse)"},{"line_number":52,"context_line":"                if rowcount \u003d\u003d 0:"},{"line_number":53,"context_line":"                    break"},{"line_number":54,"context_line":"            else:"},{"line_number":55,"context_line":"                query \u003d session.query(RevocationEvent)"},{"line_number":56,"context_line":"                query \u003d query.filter(RevocationEvent.revoked_at \u003c oldest)"},{"line_number":57,"context_line":"                query.delete(synchronize_session\u003dFalse)"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"            session.flush()"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_7b3f1a87","line":57,"range":{"start_line":54,"start_character":12,"end_line":57,"end_character":55},"updated":"2016-09-07 03:02:32.000000000","message":"won\u0027t this run unnecesarily after the while above is break ?","commit_id":"10cc460e2a41ffd6fa713c3b7cd1052b8ef41b6e"}]}
