)]}'
{"trove/cmd/manage.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"370eb0fa1720d8aba900960940d087f89421379d","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    def db_upgrade(self, version\u003dNone, repo_path\u003dNone):"},{"line_number":48,"context_line":"        self.db_api.db_upgrade(CONF, version, repo_path\u003drepo_path)"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    @deprecated(as_of\u003ddeprecated.MITAKA)"},{"line_number":51,"context_line":"    def db_downgrade(self, version, repo_path\u003dNone):"},{"line_number":52,"context_line":"        self.db_api.db_downgrade(CONF, version, repo_path\u003drepo_path)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def execute(self):"},{"line_number":55,"context_line":"        exec_method \u003d getattr(self, CONF.action.name)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a95cdbc_543952f1","side":"PARENT","line":52,"range":{"start_line":50,"start_character":4,"end_line":52,"end_character":68},"updated":"2016-10-06 19:33:20.000000000","message":"I\u0027d suggest that you take a look at \n\nhttps://review.openstack.org/#/c/163787/11/neutron/db/migration/cli.py\n\nand do something more aligned with that.\n\ni.e. instead of removing the option, throw an exception here.\n\nFor the rest of the code, removing the downgrade code and the check to make sure there is none sounds right.","commit_id":"473d360b906dba80f0f914e88b461221c1bfd5f3"}],"trove/tests/db/migrations.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"f9c9bb4ffec98f45676532887b0499411bc34739","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        py_glob \u003d os.path.join(topdir, \"trove\", \"db\", \"sqlalchemy\","},{"line_number":57,"context_line":"                               \"migrate_repo\", \"versions\", \"*.py\")"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"        missing_downgrade \u003d []"},{"line_number":60,"context_line":"        for path in glob.iglob(py_glob):"},{"line_number":61,"context_line":"            has_downgrade \u003d False"},{"line_number":62,"context_line":"            with open(path, \"r\") as f:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_1517d1ed","line":59,"range":{"start_line":59,"start_character":8,"end_line":59,"end_character":25},"updated":"2016-10-06 16:30:37.000000000","message":"it may help the reader if the variable actually represented missing_upgrade :)\n\nOr the variable name be changed to has_downgrade ...","commit_id":"1aeaafe2c611596394c89b4c5469e252916cd5e0"}]}