)]}'
{"swift/obj/replicator.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"667db1091c4b0de1ce79a1f0ac5bd733ecbf49b6","unresolved":false,"context_lines":[{"line_number":699,"context_line":"        self.suffix_hash \u003d 0"},{"line_number":700,"context_line":"        self.replication_count \u003d 0"},{"line_number":701,"context_line":"        self.last_replication_count \u003d -1"},{"line_number":702,"context_line":"        self.replication_cycle \u003d (self.replication_cycle + 1) % 10"},{"line_number":703,"context_line":"        self.partition_times \u003d []"},{"line_number":704,"context_line":"        self.my_replication_ips \u003d self._get_my_replication_ips()"},{"line_number":705,"context_line":"        self.all_devs_info \u003d set()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_ccf8d37f","line":702,"updated":"2016-12-01 18:22:12.000000000","message":"I can still change this and see no test failures e.g.\n\n  self.replication_cycle \u003d (self.replication_cycle + 1) % 11","commit_id":"0b29c1782dc8b2195ee9702505b80be92dbba693"}],"test/unit/obj/test_replicator.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"667db1091c4b0de1ce79a1f0ac5bd733ecbf49b6","unresolved":false,"context_lines":[{"line_number":315,"context_line":"        start \u003d replicator.replication_cycle"},{"line_number":316,"context_line":"        with _mock_process(process_arg_checker):"},{"line_number":317,"context_line":"            replicator.run_once()"},{"line_number":318,"context_line":"        self.assertEqual(start + 1, replicator.replication_cycle)"},{"line_number":319,"context_line":"        self.assertFalse(process_errors)"},{"line_number":320,"context_line":"        self.assertFalse(self.logger.get_lines_for_level(\u0027error\u0027))"},{"line_number":321,"context_line":"        object_replicator.http_connect \u003d was_connector"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_8ce7bbe3","line":318,"updated":"2016-12-01 18:22:12.000000000","message":"how about we repeat some cycles and check the replication_cycle wraps back to 0?","commit_id":"0b29c1782dc8b2195ee9702505b80be92dbba693"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"667db1091c4b0de1ce79a1f0ac5bd733ecbf49b6","unresolved":false,"context_lines":[{"line_number":1732,"context_line":"                      do_listdir\u003dmock_do_listdir.return_value,"},{"line_number":1733,"context_line":"                      reclaim_age\u003dself.replicator.reclaim_age)"},{"line_number":1734,"context_line":"            for job in jobs"},{"line_number":1735,"context_line":"        ]"},{"line_number":1736,"context_line":"        for job in jobs:"},{"line_number":1737,"context_line":"            set_default(self)"},{"line_number":1738,"context_line":"            ring \u003d job[\u0027policy\u0027].object_ring"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a71b18c_2cef87e9","line":1735,"updated":"2016-12-01 18:22:12.000000000","message":"+1 great improvement, but we could use a mix of results from mock_do_listdir to be really sure that the unique result of each call is passed to _get_hashes","commit_id":"0b29c1782dc8b2195ee9702505b80be92dbba693"}]}
