)]}'
{"keystone/common/sql/migrate_repo/versions/035_add_compound_revoked_token_index.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"b4447f7792c39bee97a29a9c35e898178bc1ae04","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    meta \u003d sql.MetaData()"},{"line_number":30,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":31,"context_line":"    token \u003d sql.Table(\u0027token\u0027, meta, autoload\u003dTrue)"},{"line_number":32,"context_line":"    idx \u003d sql.Index(\u0027ix_token_expires_valid\u0027, token.c.expires)"},{"line_number":33,"context_line":"    idx.drop(migrate_engine)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2BkvQ%3D","line":32,"updated":"2013-11-22 22:37:45.000000000","message":"i played with this and it works in mysql as-is, but i was surprised that it seems to have no effect if you do/don\u0027t include the token.valid column here -- explanation appreciated if anyone has one!","commit_id":"44db547b0317c11ccf52f5e6c5b97b3c47e851e9"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"0d14eb4036f8ead896912ac71b056cd0a3464218","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    meta \u003d sql.MetaData()"},{"line_number":30,"context_line":"    meta.bind \u003d migrate_engine"},{"line_number":31,"context_line":"    token \u003d sql.Table(\u0027token\u0027, meta, autoload\u003dTrue)"},{"line_number":32,"context_line":"    idx \u003d sql.Index(\u0027ix_token_expires_valid\u0027, token.c.expires)"},{"line_number":33,"context_line":"    idx.drop(migrate_engine)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Bksk%3D","line":32,"in_reply_to":"AAAATn%2F%2BkvQ%3D","updated":"2013-11-22 22:55:56.000000000","message":"That wasn\u0027t intentional.. I have uploaded a new patch with both columns listed. It may matter in some instances, especially if it is ever, for some reason, made more strict upstream.","commit_id":"44db547b0317c11ccf52f5e6c5b97b3c47e851e9"}],"keystone/token/backends/sql.py":[{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"8af2fcb41890a20082acd06bbe80db65886b194b","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    trust_id \u003d sql.Column(sql.String(64))"},{"line_number":34,"context_line":"    __table_args__ \u003d ("},{"line_number":35,"context_line":"        sql.Index(\u0027ix_token_expires\u0027, \u0027expires\u0027),"},{"line_number":36,"context_line":"        sql.Index(\u0027ix_token_valid\u0027, \u0027valid\u0027),"},{"line_number":37,"context_line":"        sql.Index(\u0027ix_token_expires_valid\u0027, \u0027expires\u0027, \u0027valid\u0027)"},{"line_number":38,"context_line":"    )"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F9vTI%3D","line":36,"updated":"2013-12-06 14:47:55.000000000","message":"It looks like we don\u0027t ever use \u0027valid\u0027 index in isolation from expires.  With the new index this could be removed.","commit_id":"dd2c80c566f20a97a22e0d7d5a514be84772a955"}]}