)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"27f552f42e2de98d632a603457d5f9cd96ffc10e","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Error logs changed for ChunkWriteTimeout"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"---All unit tests passed-----"},{"line_number":10,"context_line":"The log message phrase \u0027ChunkWriteTimeout fetching fragments\u0027"},{"line_number":11,"context_line":"implies that the timeout has occurred"},{"line_number":12,"context_line":"while getting a fragment (from the backend object server)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f3588bc_419678fd","line":9,"updated":"2023-04-26 21:40:29.000000000","message":"we wouldn\u0027t usually include this in a commit message - the patch will not merge until all the CI tests have passed, so it is implicit.","commit_id":"b974ba945b47752c9e70f7d77a0c04e67f43dadf"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"27f552f42e2de98d632a603457d5f9cd96ffc10e","unresolved":true,"context_lines":[{"line_number":11,"context_line":"implies that the timeout has occurred"},{"line_number":12,"context_line":"while getting a fragment (from the backend object server)"},{"line_number":13,"context_line":"when in fact the timeout has occurred"},{"line_number":14,"context_line":"waiting to yield the fragment to the app iter."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ic0813e6a9844da1130091d27e3dbe272ea871d11"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4ae639cf_11dd77b8","line":14,"updated":"2023-04-26 21:40:29.000000000","message":"This is good - describes *why* the change has been made","commit_id":"b974ba945b47752c9e70f7d77a0c04e67f43dadf"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"27f552f42e2de98d632a603457d5f9cd96ffc10e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"91459f21_2d0f5afb","updated":"2023-04-26 21:40:29.000000000","message":"Thanks @Shreeya for your first contribution to Swift!\n\nI might suggest cleaning up the commit message, but otherwise, assuming CI is happy, this looks great.","commit_id":"b974ba945b47752c9e70f7d77a0c04e67f43dadf"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"4b63a13c27f01b49c76dd8b8a05567a5ef51c5bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d293e845_24229367","updated":"2023-04-28 10:16:31.000000000","message":"Thanks Shreeya!","commit_id":"9dfbac7c4d600a90fa0eafcc55faf72a8daa41bf"}],"test/unit/proxy/controllers/test_obj.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"27f552f42e2de98d632a603457d5f9cd96ffc10e","unresolved":true,"context_lines":[{"line_number":4961,"context_line":"        log_lines \u003d self.app.logger.get_lines_for_level(\u0027error\u0027)"},{"line_number":4962,"context_line":"        self.assertTrue(log_lines)"},{"line_number":4963,"context_line":"        for line in log_lines:"},{"line_number":4964,"context_line":"            self.assertIn(\u0027ChunkWriteTimeout feeding fragments\u0027, line)"},{"line_number":4965,"context_line":"        # client gets a short read"},{"line_number":4966,"context_line":"        self.assertEqual(16051, len(test_data))"},{"line_number":4967,"context_line":"        self.assertEqual(8192, len(resp_body))"}],"source_content_type":"text/x-python","patch_set":1,"id":"5c11c61b_676aab04","line":4964,"updated":"2023-04-26 21:40:29.000000000","message":"oh good, we did have some test coverage! 😊","commit_id":"b974ba945b47752c9e70f7d77a0c04e67f43dadf"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"4b63a13c27f01b49c76dd8b8a05567a5ef51c5bb","unresolved":false,"context_lines":[{"line_number":4961,"context_line":"        log_lines \u003d self.app.logger.get_lines_for_level(\u0027error\u0027)"},{"line_number":4962,"context_line":"        self.assertTrue(log_lines)"},{"line_number":4963,"context_line":"        for line in log_lines:"},{"line_number":4964,"context_line":"            self.assertIn(\u0027ChunkWriteTimeout feeding fragments\u0027, line)"},{"line_number":4965,"context_line":"        # client gets a short read"},{"line_number":4966,"context_line":"        self.assertEqual(16051, len(test_data))"},{"line_number":4967,"context_line":"        self.assertEqual(8192, len(resp_body))"}],"source_content_type":"text/x-python","patch_set":1,"id":"5fc7a0ab_dea4935c","line":4964,"in_reply_to":"5c11c61b_676aab04","updated":"2023-04-28 10:16:31.000000000","message":"Ack","commit_id":"b974ba945b47752c9e70f7d77a0c04e67f43dadf"}]}