)]}'
{"test/unit/container/test_sharder.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"a09e37715b5fc1638d93c7abca4eef48e0598537","unresolved":false,"context_lines":[{"line_number":761,"context_line":"            # the listing limit"},{"line_number":762,"context_line":"            with self._mock_sharder() as sharder:"},{"line_number":763,"context_line":"                sharder._misplaced_objects(broker, node, \u0027a\u0027, \u0027c\u0027, None)"},{"line_number":764,"context_line":"        # TODO: the deleted misplaced row in fourth range should also be moved?"},{"line_number":765,"context_line":"        sharder._replicate_object.assert_has_calls("},{"line_number":766,"context_line":"            [mock.call(0, db, 0) for db in expected_shard_dbs[2:4]],"},{"line_number":767,"context_line":"            any_order\u003dTrue"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_69941645","side":"PARENT","line":764,"updated":"2017-10-05 16:24:22.000000000","message":"I\u0027m right in thinking we already had that assertion, yeah? Down at L777","commit_id":"f21831680adc211eb055aa0cf548a806c509582a"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"2bf9a745f8f097c4c715f4eaa3e4ed951766158d","unresolved":false,"context_lines":[{"line_number":761,"context_line":"            # the listing limit"},{"line_number":762,"context_line":"            with self._mock_sharder() as sharder:"},{"line_number":763,"context_line":"                sharder._misplaced_objects(broker, node, \u0027a\u0027, \u0027c\u0027, None)"},{"line_number":764,"context_line":"        # TODO: the deleted misplaced row in fourth range should also be moved?"},{"line_number":765,"context_line":"        sharder._replicate_object.assert_has_calls("},{"line_number":766,"context_line":"            [mock.call(0, db, 0) for db in expected_shard_dbs[2:4]],"},{"line_number":767,"context_line":"            any_order\u003dTrue"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_98766906","side":"PARENT","line":764,"in_reply_to":"7f515b1d_69941645","updated":"2017-10-06 08:29:00.000000000","message":"correct, the TODO was fixed here [1] but I forgot to delete the comment\n\nhttps://review.openstack.org/#/c/509496/2/test/unit/container/test_sharder.py@767","commit_id":"f21831680adc211eb055aa0cf548a806c509582a"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"2bf9a745f8f097c4c715f4eaa3e4ed951766158d","unresolved":false,"context_lines":[{"line_number":647,"context_line":"        self._check_objects(objects[:2], expected_shard_dbs[0])"},{"line_number":648,"context_line":"        self._check_objects(objects[2:5], expected_shard_dbs[1])"},{"line_number":649,"context_line":"        self._check_objects(objects[5:6], expected_shard_dbs[2])"},{"line_number":650,"context_line":"        # NB includes the deleted object"},{"line_number":651,"context_line":"        self._check_objects(objects[6:], expected_shard_dbs[3])"},{"line_number":652,"context_line":""},{"line_number":653,"context_line":"        shard_infos \u003d [ContainerBroker(shard_db_file).get_info()"},{"line_number":654,"context_line":"                       for shard_db_file in expected_shard_dbs]"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f515b1d_58c7d113","line":651,"range":{"start_line":650,"start_character":8,"end_line":651,"end_character":63},"updated":"2017-10-06 08:29:00.000000000","message":"this is the significant new assertion i.e. deleted objects[6] is now cleaved","commit_id":"e57ea584f2139a4d16974dc8ab201d823463fb6b"}]}