)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"0b603c392c5066277f9df701b993baec0738a385","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1b2e55c6_e9ce5b4f","updated":"2022-01-19 15:46:38.000000000","message":"If people are happy with the change to the NodeIter interface then I will squash this into the parent patch","commit_id":"c57f5849818fdc705e76cde625c33262aacccd0e"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"537a8ac43a3d113478ae645c68e86b1914644982","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4232d7a6_00deac10","updated":"2022-01-28 19:08:43.000000000","message":"Yeah, sure -- LGTM.","commit_id":"a50726f27a1cf76339fb23253389bc58539979b7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4133944bb60aa563ea4d99b4ac7cb9c00bb6c5b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"025a4515_4fc24073","updated":"2022-02-04 06:37:24.000000000","message":"recheck","commit_id":"a50726f27a1cf76339fb23253389bc58539979b7"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"25da6ca23b6c6dfbc07496b684b8bce55661ed36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"09ae06b2_b80d3dc5","updated":"2022-02-04 00:07:21.000000000","message":"recheck","commit_id":"a50726f27a1cf76339fb23253389bc58539979b7"}],"swift/proxy/server.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"537a8ac43a3d113478ae645c68e86b1914644982","unresolved":true,"context_lines":[{"line_number":702,"context_line":""},{"line_number":703,"context_line":"    def iter_nodes(self, ring, partition, logger, node_iter\u003dNone, policy\u003dNone):"},{"line_number":704,"context_line":"        return NodeIter(self, ring, partition, logger, node_iter\u003dnode_iter,"},{"line_number":705,"context_line":"                        policy\u003dpolicy)"},{"line_number":706,"context_line":""},{"line_number":707,"context_line":"    def exception_occurred(self, node, typ, additional_info,"},{"line_number":708,"context_line":"                           **kwargs):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9386bbd1_b86a806b","line":705,"updated":"2022-01-28 19:08:43.000000000","message":"So the one and only place we use NodeIter was already providing a logger directly? This feels a little silly... I suppose I probably only feel that way because the parent already cleaned up all the iter_nodes() callers to provide a logger.","commit_id":"a50726f27a1cf76339fb23253389bc58539979b7"}]}