)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"65c41404206eda89ad91bb5d5d7552cce0d62e9d","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"On the one hand it seems appropriate to encapsulate the incomplete"},{"line_number":13,"context_line":"chunk detection in ChunkReader; as a class, ChunkReader ought to"},{"line_number":14,"context_line":"detect short reads."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"On the other hand, I could understand an argument that the existing"},{"line_number":17,"context_line":"fix is more obvious."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0fa252eb_44eaab42","line":14,"updated":"2026-05-11 19:51:17.000000000","message":"\u003e ought to detect short reads\n\nit can only detect short reads *for this chunk* - not in general","commit_id":"06897ea36eee975d0452a618cf0750904c64babe"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"65c41404206eda89ad91bb5d5d7552cce0d62e9d","unresolved":true,"context_lines":[{"line_number":14,"context_line":"detect short reads."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"On the other hand, I could understand an argument that the existing"},{"line_number":17,"context_line":"fix is more obvious."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I4b9b1f6440c5d22af396bdc4ea8360236e1b6a11"},{"line_number":20,"context_line":"Signed-off-by: Alistair Coles \u003calistairncoles@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c8a662b3_eda1533a","line":17,"updated":"2026-05-11 19:51:17.000000000","message":"it\u0027s also the loop that had the bug; I don\u0027t like \"i can write an infinite loop here b/c the caller will raise an error if that\u0027s bad\"","commit_id":"06897ea36eee975d0452a618cf0750904c64babe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"65c41404206eda89ad91bb5d5d7552cce0d62e9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9b03f83d_70848a17","updated":"2026-05-11 19:51:17.000000000","message":"I was wrong, looks like this class does already raise `S3InputIncomplete`\n\n```\n  - ChunkReader (line 283): 3 raises — lines 305, 312, 315\n  - StreamingInput (line 343): 7 raises — lines 392, 400, 442, 454, 463, 472, 476\n```\n\n... what a mess","commit_id":"06897ea36eee975d0452a618cf0750904c64babe"}]}