)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7d7774b9266536b88b67cc9cda909f1f825dfe98","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e5c7aded_d1bf765f","updated":"2022-05-03 18:06:01.000000000","message":"All looks reasonable.","commit_id":"8d2f4d3dc9736f83ceb90f9b6f9db720c9ef2b6c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f982e1d561167acbdb111484a4a006f6d3430c16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"12347634_7d43ccf7","in_reply_to":"e5c7aded_d1bf765f","updated":"2022-05-04 12:55:11.000000000","message":"Thanks for your review !","commit_id":"8d2f4d3dc9736f83ceb90f9b6f9db720c9ef2b6c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"5f509cd6e5c6e5a4bac82583e46485b3dab51cc4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"19c50cf7_f9d90d46","updated":"2022-05-13 10:33:27.000000000","message":"I\u0027ll approve this now becaus this would block py2 support removal in Swift.","commit_id":"132d3509f2b162befd460c2d0ea4d6044b3565a1"}],"storlets/agent/common/utils.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7d7774b9266536b88b67cc9cda909f1f825dfe98","unresolved":true,"context_lines":[{"line_number":15,"context_line":"import logging"},{"line_number":16,"context_line":"from logging.handlers import SysLogHandler"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# TODO(takashi): Make the following parameters configurable"},{"line_number":19,"context_line":"DEFAULT_PY3 \u003d 3.8"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"37788e96_273a8ce9","line":18,"range":{"start_line":18,"start_character":36,"end_line":18,"end_character":46},"updated":"2022-05-03 18:06:01.000000000","message":"nit: parameter","commit_id":"8d2f4d3dc9736f83ceb90f9b6f9db720c9ef2b6c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f982e1d561167acbdb111484a4a006f6d3430c16","unresolved":false,"context_lines":[{"line_number":15,"context_line":"import logging"},{"line_number":16,"context_line":"from logging.handlers import SysLogHandler"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"# TODO(takashi): Make the following parameters configurable"},{"line_number":19,"context_line":"DEFAULT_PY3 \u003d 3.8"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"cb43d33d_133ec4c0","line":18,"range":{"start_line":18,"start_character":36,"end_line":18,"end_character":46},"in_reply_to":"37788e96_273a8ce9","updated":"2022-05-04 12:55:11.000000000","message":"Done","commit_id":"8d2f4d3dc9736f83ceb90f9b6f9db720c9ef2b6c"}],"storlets/gateway/gateways/docker/gateway.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"7d7774b9266536b88b67cc9cda909f1f825dfe98","unresolved":true,"context_lines":[{"line_number":133,"context_line":"            except ValueError:"},{"line_number":134,"context_line":"                raise ValueError(\u0027Language-Version is invalid\u0027)"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"            # TODO(takashi): Drop Py2 support"},{"line_number":137,"context_line":"            if version not in [3, DEFAULT_PY3]:"},{"line_number":138,"context_line":"                # TODO(kota_): more strict version check should be nice."},{"line_number":139,"context_line":"                raise ValueError(\u0027Not supported version specified\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"64fa5d04_02940194","line":136,"updated":"2022-05-03 18:06:01.000000000","message":"Can probably drop this comment.","commit_id":"8d2f4d3dc9736f83ceb90f9b6f9db720c9ef2b6c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f982e1d561167acbdb111484a4a006f6d3430c16","unresolved":false,"context_lines":[{"line_number":133,"context_line":"            except ValueError:"},{"line_number":134,"context_line":"                raise ValueError(\u0027Language-Version is invalid\u0027)"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"            # TODO(takashi): Drop Py2 support"},{"line_number":137,"context_line":"            if version not in [3, DEFAULT_PY3]:"},{"line_number":138,"context_line":"                # TODO(kota_): more strict version check should be nice."},{"line_number":139,"context_line":"                raise ValueError(\u0027Not supported version specified\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ceaddad_13c1e921","line":136,"in_reply_to":"64fa5d04_02940194","updated":"2022-05-04 12:55:11.000000000","message":"Done","commit_id":"8d2f4d3dc9736f83ceb90f9b6f9db720c9ef2b6c"}]}
