)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7273db35ced5993257b9d1a1b0d535aa06495b31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"35ad4da9_d44bd0f1","updated":"2025-12-01 03:19:27.000000000","message":"I hate to make you restack the whole chain, but at the same time, splitting something like `timestamps[2].internal` across multiple lines just bugs me. Looks good otherwise.","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"}],"test/unit/container/test_backend.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7273db35ced5993257b9d1a1b0d535aa06495b31","unresolved":true,"context_lines":[{"line_number":1846,"context_line":"        hsh \u003d hash_path(acct, cont)"},{"line_number":1847,"context_line":"        db_file \u003d \"%s.db\" % hsh"},{"line_number":1848,"context_line":"        db_path \u003d os.path.join(tempdir, db_file)"},{"line_number":1849,"context_line":"        ts_put \u003d next(self.ts)"},{"line_number":1850,"context_line":""},{"line_number":1851,"context_line":"        broker \u003d ContainerBroker(db_path, account\u003dacct, container\u003dcont)"},{"line_number":1852,"context_line":"        broker.initialize(ts_put.internal, 0)"}],"source_content_type":"text/x-python","patch_set":6,"id":"bfe33677_55ff7709","line":1849,"updated":"2025-12-01 03:19:27.000000000","message":"Not `put_ts`, to match the `delete_ts` later?","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c053c9716af6a201a3bf230a98ef2d53b92f45b9","unresolved":false,"context_lines":[{"line_number":1846,"context_line":"        hsh \u003d hash_path(acct, cont)"},{"line_number":1847,"context_line":"        db_file \u003d \"%s.db\" % hsh"},{"line_number":1848,"context_line":"        db_path \u003d os.path.join(tempdir, db_file)"},{"line_number":1849,"context_line":"        ts_put \u003d next(self.ts)"},{"line_number":1850,"context_line":""},{"line_number":1851,"context_line":"        broker \u003d ContainerBroker(db_path, account\u003dacct, container\u003dcont)"},{"line_number":1852,"context_line":"        broker.initialize(ts_put.internal, 0)"}],"source_content_type":"text/x-python","patch_set":6,"id":"19d78d70_92e2d7e3","line":1849,"in_reply_to":"bfe33677_55ff7709","updated":"2025-12-01 12:19:04.000000000","message":"good call","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7273db35ced5993257b9d1a1b0d535aa06495b31","unresolved":true,"context_lines":[{"line_number":3621,"context_line":"        snowman \u003d u\u0027\\N{SNOWMAN}\u0027"},{"line_number":3622,"context_line":"        broker1 \u003d ContainerBroker(self.get_db_path(), account\u003d\u0027a\u0027,"},{"line_number":3623,"context_line":"                                  container\u003d\u0027c\u0027)"},{"line_number":3624,"context_line":"        timestamps \u003d [next(self.ts) for _ in range(5)]"},{"line_number":3625,"context_line":"        broker1.initialize(timestamps[1].internal, 0)"},{"line_number":3626,"context_line":"        id \u003d broker1.get_info()[\u0027id\u0027]"},{"line_number":3627,"context_line":"        broker2 \u003d ContainerBroker(self.get_db_path(),"}],"source_content_type":"text/x-python","patch_set":6,"id":"00a1dc6d_0eacd27b","line":3624,"updated":"2025-12-01 03:19:27.000000000","message":"Not using `timestamps[0]`? I suppose it makes for an easier substitution...","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c053c9716af6a201a3bf230a98ef2d53b92f45b9","unresolved":false,"context_lines":[{"line_number":3621,"context_line":"        snowman \u003d u\u0027\\N{SNOWMAN}\u0027"},{"line_number":3622,"context_line":"        broker1 \u003d ContainerBroker(self.get_db_path(), account\u003d\u0027a\u0027,"},{"line_number":3623,"context_line":"                                  container\u003d\u0027c\u0027)"},{"line_number":3624,"context_line":"        timestamps \u003d [next(self.ts) for _ in range(5)]"},{"line_number":3625,"context_line":"        broker1.initialize(timestamps[1].internal, 0)"},{"line_number":3626,"context_line":"        id \u003d broker1.get_info()[\u0027id\u0027]"},{"line_number":3627,"context_line":"        broker2 \u003d ContainerBroker(self.get_db_path(),"}],"source_content_type":"text/x-python","patch_set":6,"id":"446d4b25_eb0e1c9b","line":3624,"in_reply_to":"00a1dc6d_0eacd27b","updated":"2025-12-01 12:19:04.000000000","message":"you guessed it! and easier review perhaps,  but now you\u0027re familiar I\u0027ll change the indices 😊","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7273db35ced5993257b9d1a1b0d535aa06495b31","unresolved":true,"context_lines":[{"line_number":3628,"context_line":"                                  account\u003d\u0027a\u0027, container\u003d\u0027c\u0027)"},{"line_number":3629,"context_line":"        broker2.initialize(timestamps[1].internal, 0)"},{"line_number":3630,"context_line":"        broker1.put_object(snowman, timestamps[2]"},{"line_number":3631,"context_line":"                           .internal, 0,"},{"line_number":3632,"context_line":"                           \u0027text/plain\u0027, \u0027d41d8cd98f00b204e9800998ecf8427e\u0027)"},{"line_number":3633,"context_line":"        broker1.put_object(\u0027b\u0027, timestamps[3]"},{"line_number":3634,"context_line":"                           .internal, 0,"}],"source_content_type":"text/x-python","patch_set":6,"id":"83e0892a_fcdd197d","line":3631,"updated":"2025-12-01 03:19:27.000000000","message":"Are we right at the line-length limit or something? I didn\u0027t think we were **so** close, but still, I\u0027d rather see\n```\n        broker1.put_object(snowman,\n                           timestamps[2].internal, 0,\n                           \u0027text/plain\u0027, \u0027d41d8cd98f00b204e9800998ecf8427e\u0027)\n```\nor\n```\n        broker1.put_object(\n            snowman, timestamps[2].internal, 0,\n            \u0027text/plain\u0027, \u0027d41d8cd98f00b204e9800998ecf8427e\u0027)\n```\nor\n```\n        broker1.put_object(\n            snowman,\n            timestamps[2].internal,\n            0,\n            \u0027text/plain\u0027,\n            \u0027d41d8cd98f00b204e9800998ecf8427e\u0027,\n        )\n```\nthan split an attr-access from its object...\n\n(With similar ick in the rest of this test.)","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c053c9716af6a201a3bf230a98ef2d53b92f45b9","unresolved":false,"context_lines":[{"line_number":3628,"context_line":"                                  account\u003d\u0027a\u0027, container\u003d\u0027c\u0027)"},{"line_number":3629,"context_line":"        broker2.initialize(timestamps[1].internal, 0)"},{"line_number":3630,"context_line":"        broker1.put_object(snowman, timestamps[2]"},{"line_number":3631,"context_line":"                           .internal, 0,"},{"line_number":3632,"context_line":"                           \u0027text/plain\u0027, \u0027d41d8cd98f00b204e9800998ecf8427e\u0027)"},{"line_number":3633,"context_line":"        broker1.put_object(\u0027b\u0027, timestamps[3]"},{"line_number":3634,"context_line":"                           .internal, 0,"}],"source_content_type":"text/x-python","patch_set":6,"id":"0ba80426_001e03d7","line":3631,"in_reply_to":"83e0892a_fcdd197d","updated":"2025-12-01 12:19:04.000000000","message":"oh no, that\u0027s awful, IDK how that got by me, probably I was using multi-cursor editing and didn\u0027t notice the line wrap. It\u0027s not even necessary.","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7273db35ced5993257b9d1a1b0d535aa06495b31","unresolved":true,"context_lines":[{"line_number":6969,"context_line":"        except AssertionError as e:"},{"line_number":6970,"context_line":"            msg \u003d (\u0027Scenario: existing %s, new_item %s, expected %s.\u0027"},{"line_number":6971,"context_line":"                   % scenario)"},{"line_number":6972,"context_line":"            msg \u003d \u0027%s Failed with: %s\u0027 % (msg, e)"},{"line_number":6973,"context_line":"            raise AssertionError(msg)"},{"line_number":6974,"context_line":""},{"line_number":6975,"context_line":"    def test_update_new_item_from_existing(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"ec1edc8b_08cc3d8b","line":6972,"updated":"2025-12-01 03:19:27.000000000","message":"Whoa! A belated py2-\u003e3 fix??","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"c053c9716af6a201a3bf230a98ef2d53b92f45b9","unresolved":false,"context_lines":[{"line_number":6969,"context_line":"        except AssertionError as e:"},{"line_number":6970,"context_line":"            msg \u003d (\u0027Scenario: existing %s, new_item %s, expected %s.\u0027"},{"line_number":6971,"context_line":"                   % scenario)"},{"line_number":6972,"context_line":"            msg \u003d \u0027%s Failed with: %s\u0027 % (msg, e)"},{"line_number":6973,"context_line":"            raise AssertionError(msg)"},{"line_number":6974,"context_line":""},{"line_number":6975,"context_line":"    def test_update_new_item_from_existing(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"242f18cc_514c7bad","line":6972,"in_reply_to":"ec1edc8b_08cc3d8b","updated":"2025-12-01 12:19:04.000000000","message":"😄","commit_id":"755d9d41e9868f428afcdace628b6437bcbdc224"}]}