)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c62dc4bff4c97f28700736c0787954f37db49a9d","unresolved":true,"context_lines":[{"line_number":16,"context_line":"Python class name."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"This extracts the controller-side pattern from the multipart"},{"line_number":19,"context_line":"feature work without pulling in the native MPU implementation."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: I9bbbe8fc0a8849a4d499104754e1274fadc9167a"},{"line_number":22,"context_line":"Signed-off-by: Clay Gerrard \u003cclay.gerrard@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d3434270_996b7bfc","line":19,"updated":"2026-07-21 12:12:19.000000000","message":"for posterity, this comment is referring to https://review.opendev.org/c/openstack/swift/+/997730 and https://review.opendev.org/c/openstack/swift/+/997779","commit_id":"5100a2fc9931e48acc9f7a7d295ccab4afebeaaa"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"9217d2502e8b3c85d858e0ae53a79f414dd7c628","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"60387348_1868f028","updated":"2026-07-21 08:15:54.000000000","message":"LGTM, clean refactoring.","commit_id":"5100a2fc9931e48acc9f7a7d295ccab4afebeaaa"}],"swift/common/middleware/s3api/s3request.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c62dc4bff4c97f28700736c0787954f37db49a9d","unresolved":true,"context_lines":[{"line_number":1819,"context_line":""},{"line_number":1820,"context_line":"    @property"},{"line_number":1821,"context_line":"    def controller_name(self):"},{"line_number":1822,"context_line":"        return self.controller.name()"},{"line_number":1823,"context_line":""},{"line_number":1824,"context_line":"    @property"},{"line_number":1825,"context_line":"    def controller(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fae4ea54_47d12a08","line":1822,"updated":"2026-07-21 12:12:19.000000000","message":"AFAICT this property, and the controller method, is now only used in two places where it would be more explicit, perhaps better even, to just use a literal string - how about we just remove it?\n\nUPDATE: this happens in the next patch 😀","commit_id":"5100a2fc9931e48acc9f7a7d295ccab4afebeaaa"}]}
