)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"41027c38e95185e5a25b5dd55794004450703b1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ccf48df8_ac731fdc","updated":"2023-04-20 23:44:15.000000000","message":"Looks better without the additional sleep().","commit_id":"e4ee6749707cb004297ee04af16c1547073cbb9d"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7abfa6855854d9135f91c49b5eebd8d9a703b4fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0e99658e_9f743308","updated":"2023-04-24 14:15:27.000000000","message":"I compared relocated functions and can see no behavioural change. Big win for grok\u0027ing the code.","commit_id":"44183e660818929860672dd0aa234f3e3245fa87"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"a669ebd4daa461155c23a5b0c64c9d23a7e4579b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5b2ecd32_c23ebd54","updated":"2023-04-24 14:19:26.000000000","message":"It looks like a rebase went wrong, Zuul is unable to merge this.","commit_id":"44183e660818929860672dd0aa234f3e3245fa87"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"bfcbfab95b1913f4fa699a6706fe3883bf20573e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4b39b582_5b1ec691","updated":"2023-04-24 14:28:20.000000000","message":"Should be fixed now. That will teach me to try to be helpful by fixing a nit in the parent patch!","commit_id":"74fc18d90df38efd12a75e64dc9645156a7f65a6"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"8c4b2a2126ea31637e75982f11b38b57d1aa662a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2285aa1e_74befbf3","updated":"2023-04-24 17:29:15.000000000","message":"recheck\n\nhttps://bugs.launchpad.net/swift/+bug/2017021","commit_id":"74fc18d90df38efd12a75e64dc9645156a7f65a6"}],"swift/proxy/controllers/obj.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7abfa6855854d9135f91c49b5eebd8d9a703b4fa","unresolved":true,"context_lines":[{"line_number":2621,"context_line":"        return it"},{"line_number":2622,"context_line":""},{"line_number":2623,"context_line":"    def get_next_doc_part(self):"},{"line_number":2624,"context_line":"        node_timeout \u003d self.app.recoverable_node_timeout"},{"line_number":2625,"context_line":""},{"line_number":2626,"context_line":"        while True:"},{"line_number":2627,"context_line":"            # the loop here is to resume if trying to parse"}],"source_content_type":"text/x-python","patch_set":3,"id":"5792f3f4_7b1b7520","line":2624,"updated":"2023-04-24 14:15:27.000000000","message":"nit: only used once in this method","commit_id":"44183e660818929860672dd0aa234f3e3245fa87"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"a669ebd4daa461155c23a5b0c64c9d23a7e4579b","unresolved":true,"context_lines":[{"line_number":2621,"context_line":"        return it"},{"line_number":2622,"context_line":""},{"line_number":2623,"context_line":"    def get_next_doc_part(self):"},{"line_number":2624,"context_line":"        node_timeout \u003d self.app.recoverable_node_timeout"},{"line_number":2625,"context_line":""},{"line_number":2626,"context_line":"        while True:"},{"line_number":2627,"context_line":"            # the loop here is to resume if trying to parse"}],"source_content_type":"text/x-python","patch_set":3,"id":"fbfe4c0e_30decfb8","line":2624,"in_reply_to":"5792f3f4_7b1b7520","updated":"2023-04-24 14:19:26.000000000","message":"I don\u0027t feel strongly in this case, because an extra variable does not eliminate the line continuation. If the result fits into 79 characters, I suppose it can be moved to the call site.","commit_id":"44183e660818929860672dd0aa234f3e3245fa87"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7abfa6855854d9135f91c49b5eebd8d9a703b4fa","unresolved":true,"context_lines":[{"line_number":2660,"context_line":"                        read_chunk_size\u003dself.app.object_chunk_size)"},{"line_number":2661,"context_line":""},{"line_number":2662,"context_line":"    def iter_bytes_from_response_part(self, part_file, nbytes):"},{"line_number":2663,"context_line":"        client_chunk_size \u003d self.client_chunk_size"},{"line_number":2664,"context_line":"        node_timeout \u003d self.app.recoverable_node_timeout"},{"line_number":2665,"context_line":"        nchunks \u003d 0"},{"line_number":2666,"context_line":"        buf \u003d b\u0027\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"288fc5d9_dbfb3772","line":2663,"updated":"2023-04-24 14:15:27.000000000","message":"nit: could be located adjacent to where it is used later in method","commit_id":"44183e660818929860672dd0aa234f3e3245fa87"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"7abfa6855854d9135f91c49b5eebd8d9a703b4fa","unresolved":true,"context_lines":[{"line_number":2661,"context_line":""},{"line_number":2662,"context_line":"    def iter_bytes_from_response_part(self, part_file, nbytes):"},{"line_number":2663,"context_line":"        client_chunk_size \u003d self.client_chunk_size"},{"line_number":2664,"context_line":"        node_timeout \u003d self.app.recoverable_node_timeout"},{"line_number":2665,"context_line":"        nchunks \u003d 0"},{"line_number":2666,"context_line":"        buf \u003d b\u0027\u0027"},{"line_number":2667,"context_line":"        part_file \u003d ByteCountEnforcer(part_file, nbytes)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3929d0dc_e560c4b7","line":2664,"updated":"2023-04-24 14:15:27.000000000","message":"nit: only used once in this method","commit_id":"44183e660818929860672dd0aa234f3e3245fa87"}]}