)]}'
{"test/unit/common/test_db_replicator.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"83f8226faed7c94a21b0bc2c459058429a307d5a","unresolved":false,"context_lines":[{"line_number":1120,"context_line":"                                          mount_check\u003dFalse)"},{"line_number":1121,"context_line":"        with unit.mock_check_drive(isdir\u003dTrue):"},{"line_number":1122,"context_line":"            response \u003d rpc.dispatch((\u0027a\u0027,), \u0027arg\u0027)"},{"line_number":1123,"context_line":"        self.assertEqual(b\u0027Invalid object type\u0027, response.body)"},{"line_number":1124,"context_line":"        self.assertEqual(400, response.status_int)"},{"line_number":1125,"context_line":""},{"line_number":1126,"context_line":"    def test_dispatch_drive_not_mounted(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_c280d7c1","line":1123,"updated":"2018-11-01 00:48:15.000000000","message":"It\u0027s impressive that so little needed to change -- to the point that I\u0027m a little skeptical that we\u0027re testing DB replication sufficiently :-/\n\nLike, why didn\u0027t we need to pick up the change at https://review.openstack.org/#/c/613505/2/swift/common/db_replicator.py ??","commit_id":"654187e1feff122ccf9a6d3dead6c86b4f8eb97c"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"70efc67a0ea07314f68d98f72e180199766f30ff","unresolved":false,"context_lines":[{"line_number":1120,"context_line":"                                          mount_check\u003dFalse)"},{"line_number":1121,"context_line":"        with unit.mock_check_drive(isdir\u003dTrue):"},{"line_number":1122,"context_line":"            response \u003d rpc.dispatch((\u0027a\u0027,), \u0027arg\u0027)"},{"line_number":1123,"context_line":"        self.assertEqual(b\u0027Invalid object type\u0027, response.body)"},{"line_number":1124,"context_line":"        self.assertEqual(400, response.status_int)"},{"line_number":1125,"context_line":""},{"line_number":1126,"context_line":"    def test_dispatch_drive_not_mounted(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_532b377b","line":1123,"in_reply_to":"3f79a3b5_c280d7c1","updated":"2018-11-01 02:43:50.000000000","message":"I have concerns too, but that cleanup did absolutely nothing. It\u0027s a vestige of the time when swift.common.utils did six.moves-like thing and pulled json or cjson. Now it\u0027s plain old json, so all the old code did was re-importation of the stock json. The cleanup merely removes the indirection.","commit_id":"654187e1feff122ccf9a6d3dead6c86b4f8eb97c"}]}