)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"5e04febfffa36278d76f3df406b308b5326dc274","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"31d06988_8e52086b","updated":"2026-09-01 23:07:56.000000000","message":"i\u0027m pretty sure we could do this and it would look great on master.","commit_id":"2cca5e7ebc461a02aa71a83a101121559f68703f"}],"swift/proxy/controllers/obj.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"5e04febfffa36278d76f3df406b308b5326dc274","unresolved":true,"context_lines":[{"line_number":258,"context_line":"                primary_nodes, self.app.request_node_count("},{"line_number":259,"context_line":"                    len(primary_nodes)), policy\u003dpolicy)"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        primary_nodes \u003d ring.get_part_nodes(partition)"},{"line_number":262,"context_line":"        handoff_nodes \u003d ring.get_more_nodes(partition)"},{"line_number":263,"context_line":"        all_nodes \u003d itertools.chain(primary_nodes, handoff_nodes)"},{"line_number":264,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"46d21d1e_37c13e26","line":261,"updated":"2026-09-01 23:07:56.000000000","message":"boom! right here write-affinity aware `PUT` we\u0027re already building `primary_nodes` in-order to consturct the re-ordered `node_iter\u003d` and then on master `NodeIter` has to get `primary_nodes` *again* JUST to figure out `node_count`!","commit_id":"2cca5e7ebc461a02aa71a83a101121559f68703f"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"5e04febfffa36278d76f3df406b308b5326dc274","unresolved":true,"context_lines":[{"line_number":292,"context_line":"        return NodeIter("},{"line_number":293,"context_line":"            \u0027object\u0027, self.app, ring, partition, self.logger, request,"},{"line_number":294,"context_line":"            primary_nodes, self.app.request_node_count(len(primary_nodes)),"},{"line_number":295,"context_line":"            node_iter\u003dnode_iter, policy\u003dpolicy)"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"    def GETorHEAD(self, req):"},{"line_number":298,"context_line":"        \"\"\"Handle HTTP GET or HEAD requests.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"410e3d0b_b9d028ac","line":295,"updated":"2026-09-01 23:07:56.000000000","message":"this isn\u0027t some new intermediary refactor that only shows up in this patch chain - we\u0027re no where near write affinity - this would look equally dumb on master and making NodeIter require primary_nodes and app-config request_node_count as posistional arguments is JUST a design win!","commit_id":"2cca5e7ebc461a02aa71a83a101121559f68703f"}]}
