)]}'
{"swift/account/reaper.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"ba9d6d1369d48c58f250f23566f0b65f4812855f","unresolved":false,"context_lines":[{"line_number":172,"context_line":"            for container_shard, node in enumerate(nodes):"},{"line_number":173,"context_line":"                if is_local_device(self.myips, self.bind_port,"},{"line_number":174,"context_line":"                                   node[\u0027ip\u0027], node[\u0027port\u0027]) and \\"},{"line_number":175,"context_line":"                                  (device \u003d\u003d node[\u0027device\u0027]):"},{"line_number":176,"context_line":"                    break"},{"line_number":177,"context_line":"            else:"},{"line_number":178,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aed3d3a_333e00b9","line":175,"updated":"2016-02-25 17:53:45.000000000","message":"so this is specifically for the case when multiple replicas of the account.db are on multiple devices on the same machine\n\n... so device \u003d\u003d node[\u0027device\u0027] makes sense, make sure this node in the ring that matches this devices ip\u0027s also matches the name of the device we got from the listdir - otherwise container_shard is wrong and at least one of the devices ends up skipping the wrong containers when the reap rolls over that device on disk.\n\nNot sure about when self.bind_port is None exactly - that may only effect servers_per_port - but I think it\u0027d be reasonable to leave that in for \"future\" proofing if we ever add servers_per_port to account/containers.","commit_id":"44bdec93386d5d13ea98088eac296d39fd65dc96"},{"author":{"_account_id":12050,"name":"Charles Hsu","email":"charles0126@gmail.com","username":"charz"},"change_message_id":"54ba8b82b21fd56f60ae22c6049bc7ddd6b83c70","unresolved":false,"context_lines":[{"line_number":172,"context_line":"            for container_shard, node in enumerate(nodes):"},{"line_number":173,"context_line":"                if is_local_device(self.myips, self.bind_port,"},{"line_number":174,"context_line":"                                   node[\u0027ip\u0027], node[\u0027port\u0027]) and \\"},{"line_number":175,"context_line":"                                  (device \u003d\u003d node[\u0027device\u0027]):"},{"line_number":176,"context_line":"                    break"},{"line_number":177,"context_line":"            else:"},{"line_number":178,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":3,"id":"7af24918_c09fec5d","line":175,"in_reply_to":"9aed3d3a_333e00b9","updated":"2016-03-01 15:24:54.000000000","message":"Okay, you\u0027re right. Just add device \u003d\u003d node[\u0027device\u0027] and leave the `port` for someone.","commit_id":"44bdec93386d5d13ea98088eac296d39fd65dc96"}]}
