)]}'
{"keystone/tests/test_sql_upgrade.py":[{"author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"change_message_id":"1e35183202f5cdc49c8e7fc5b9da4c44463327d0","unresolved":false,"context_lines":[{"line_number":2047,"context_line":"        # downgrade"},{"line_number":2048,"context_line":"        self.upgrade(41)"},{"line_number":2049,"context_line":"        session \u003d self.Session()"},{"line_number":2050,"context_line":"        trust_table \u003d sqlalchemy.Table("},{"line_number":2051,"context_line":"            \u0027trust\u0027, self.metadata, autoload\u003dTrue)"},{"line_number":2052,"context_line":"        limited_trust \u003d {"},{"line_number":2053,"context_line":"            \u0027id\u0027: uuid.uuid4().hex,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2FCPM%3D","side":"PARENT","line":2050,"updated":"2014-03-10 14:43:55.000000000","message":"Make sure this actually works with the live databases.  It was added as they need to load the table for reference sake.  I think it is necessary (maybe the variable name is not, but the call to .Table is","commit_id":"58bb5e97ffab23c8323ad9ff8f6c4989d45a3355"},{"author":{"_account_id":7186,"name":"Matthieu Huin","email":"mhuin@redhat.com","username":"mhu"},"change_message_id":"3be88e70fb90c81022f2f05a1f55b8fa70dab553","unresolved":false,"context_lines":[{"line_number":2047,"context_line":"        # downgrade"},{"line_number":2048,"context_line":"        self.upgrade(41)"},{"line_number":2049,"context_line":"        session \u003d self.Session()"},{"line_number":2050,"context_line":"        trust_table \u003d sqlalchemy.Table("},{"line_number":2051,"context_line":"            \u0027trust\u0027, self.metadata, autoload\u003dTrue)"},{"line_number":2052,"context_line":"        limited_trust \u003d {"},{"line_number":2053,"context_line":"            \u0027id\u0027: uuid.uuid4().hex,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2F1h4%3D","side":"PARENT","line":2050,"in_reply_to":"AAAAVn%2F%2FCPM%3D","updated":"2014-03-13 13:57:36.000000000","message":"The tests still pass, I\u0027d be tempted to say this can be safely removed.","commit_id":"58bb5e97ffab23c8323ad9ff8f6c4989d45a3355"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"05eea1afce1b50286c3d5119760f2ab4a6a21838","unresolved":false,"context_lines":[{"line_number":2047,"context_line":"        # downgrade"},{"line_number":2048,"context_line":"        self.upgrade(41)"},{"line_number":2049,"context_line":"        session \u003d self.Session()"},{"line_number":2050,"context_line":"        trust_table \u003d sqlalchemy.Table("},{"line_number":2051,"context_line":"            \u0027trust\u0027, self.metadata, autoload\u003dTrue)"},{"line_number":2052,"context_line":"        limited_trust \u003d {"},{"line_number":2053,"context_line":"            \u0027id\u0027: uuid.uuid4().hex,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2B834%3D","side":"PARENT","line":2050,"in_reply_to":"AAAAVn%2F%2FCPM%3D","updated":"2014-03-11 03:40:59.000000000","message":"see https://review.openstack.org/#/c/56243/26/keystone/tests/test_sql_upgrade.py\n\nDuring the next table loaded at line 2079 there is no any change on the table. I think this line is a small careless mistake.:)","commit_id":"58bb5e97ffab23c8323ad9ff8f6c4989d45a3355"}]}