)]}'
{"swift/obj/replicator.py":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"a3d88bf5831e85401a7fa573458ed19dcb1dc411","unresolved":false,"context_lines":[{"line_number":211,"context_line":"            error_long \u003d _(\"Killing long-running rsync: %s\") % str(args)"},{"line_number":212,"context_line":"            if self.rsync_error_log_line_length:"},{"line_number":213,"context_line":"                error_long \u003d error_long[:self.rsync_error_log_line_length]"},{"line_number":214,"context_line":"            self.logger.error(error_long)"},{"line_number":215,"context_line":"            if proc:"},{"line_number":216,"context_line":"                proc.kill()"},{"line_number":217,"context_line":"                try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf659307_c2ab4c85","line":214,"updated":"2018-04-13 08:20:01.000000000","message":"Nice. Now that were doing the same thing twice I wonder if we should refactor it out. it might make it easier to use if we ever need to add it to any other rsync log lines.\n\nMaybe something like (quickly hacked this from master): http://paste.openstack.org/show/719126/","commit_id":"ff36e181ed63628116eaad210bf85deb0d323ff7"}],"test/unit/obj/test_replicator.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"6fa1302a54fdb712ab71b5f69f2a4c45ac825dcd","unresolved":false,"context_lines":[{"line_number":2106,"context_line":"            ])"},{"line_number":2107,"context_line":"        self.assertEqual(len(mock_procs), 2)"},{"line_number":2108,"context_line":""},{"line_number":2109,"context_line":"    def test_limit_rsync_log(self):"},{"line_number":2110,"context_line":"        def do_test(length_limit, log_line, expected):"},{"line_number":2111,"context_line":"            self.replicator.rsync_error_log_line_length \u003d length_limit"},{"line_number":2112,"context_line":"            result \u003d self.replicator._limit_rsync_log(log_line)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_aa9d40bc","line":2109,"range":{"start_line":2109,"start_character":8,"end_line":2109,"end_character":28},"updated":"2018-05-22 12:01:30.000000000","message":"this is good, but there is not test to verify that _limit_rsync_log is actually called from _rsync\n\ne.g. https://review.openstack.org/569968","commit_id":"2307886efebee4874c6606879623ad75f9f34d34"}]}