)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c0f8e7e9f921b4da074442fcef458aa3215b276c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dolph Mathews \u003cdolph.mathews@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-25 19:58:30 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Let upgrade tests control all 4 repositories at once"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I6be5e37fac9a40e405158e09b3af47c7325de524"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa7ab95a_27ee6eec","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":52},"updated":"2016-08-29 04:14:12.000000000","message":"and refactor a whole bunch of metadata code :)","commit_id":"f66077f356538736f875e15b56d25ee876fbb884"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"6586114cd82b9e3631e9bf42fe5d859077b258d5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dolph Mathews \u003cdolph.mathews@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-25 19:58:30 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Let upgrade tests control all 4 repositories at once"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I6be5e37fac9a40e405158e09b3af47c7325de524"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa7ab95a_dd1e436f","line":7,"in_reply_to":"fa7ab95a_27ee6eec","updated":"2016-08-29 19:03:01.000000000","message":"Yeah, in my head that was all in service of keeping the correct object ownership in place when moving from one repository to four. The engine needs to be shared, the metadata needs to be constantly updated, etc.","commit_id":"f66077f356538736f875e15b56d25ee876fbb884"}],"keystone/tests/unit/test_sql_upgrade.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"bac1ce79199de63c6e8d2e41cb671079f864f592","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        self.repos[LEGACY_REPO].upgrade(*args, **kwargs)"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"    def get_max_version(self, repository):"},{"line_number":250,"context_line":"        return self.repos[repository].max_version"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    def expand(self, *args, **kwargs):"},{"line_number":253,"context_line":"        \"\"\"Expand database schema.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_623456de","line":250,"updated":"2016-08-25 19:54:33.000000000","message":"We can remove this since we are just calling directly into the MigrationRepository class to get the max_version","commit_id":"6a914d46f3184f9982d78b5a59673b54ea871aee"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"93eee7db59e027576de98ecad2fdaab238fe43b9","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        self.repos[LEGACY_REPO].upgrade(*args, **kwargs)"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"    def get_max_version(self, repository):"},{"line_number":250,"context_line":"        return self.repos[repository].max_version"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    def expand(self, *args, **kwargs):"},{"line_number":253,"context_line":"        \"\"\"Expand database schema.\"\"\""}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_62c8d675","line":250,"in_reply_to":"fa7ab95a_623456de","updated":"2016-08-25 19:55:33.000000000","message":"++","commit_id":"6a914d46f3184f9982d78b5a59673b54ea871aee"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"93eee7db59e027576de98ecad2fdaab238fe43b9","unresolved":false,"context_lines":[{"line_number":263,"context_line":""},{"line_number":264,"context_line":"    @property"},{"line_number":265,"context_line":"    def metadata(self):"},{"line_number":266,"context_line":"        \"\"\"Pass the legacy metadata reference through for legacy repo tests.\"\"\""},{"line_number":267,"context_line":"        return sqlalchemy.MetaData(self.engine)"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    def select_table(self, name):"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_02b0eaf3","line":266,"updated":"2016-08-25 19:55:33.000000000","message":"docstr is outdated","commit_id":"6a914d46f3184f9982d78b5a59673b54ea871aee"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"bac1ce79199de63c6e8d2e41cb671079f864f592","unresolved":false,"context_lines":[{"line_number":264,"context_line":"    @property"},{"line_number":265,"context_line":"    def metadata(self):"},{"line_number":266,"context_line":"        \"\"\"Pass the legacy metadata reference through for legacy repo tests.\"\"\""},{"line_number":267,"context_line":"        return sqlalchemy.MetaData(self.engine)"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    def select_table(self, name):"},{"line_number":270,"context_line":"        table \u003d sqlalchemy.Table(name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_22e7ae13","line":267,"updated":"2016-08-25 19:54:33.000000000","message":"Oh....","commit_id":"6a914d46f3184f9982d78b5a59673b54ea871aee"}]}