)]}'
{"smaug/services/protection/checkpoint.py":[{"author":{"_account_id":21648,"name":"zhangshuai","email":"446077695@qq.com","username":"zhangshuai"},"change_message_id":"4d21a9abec13675552a03679225816851ceeb2df","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            \"status\": self.status,"},{"line_number":48,"context_line":"            \"protection_plan\": self.protection_plan,"},{"line_number":49,"context_line":"            \"project_id\": self.project_id,"},{"line_number":50,"context_line":"            \"provider_id\": self.provider_id,"},{"line_number":51,"context_line":"        }"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_d8e2202f","line":50,"range":{"start_line":50,"start_character":12,"end_line":50,"end_character":44},"updated":"2016-07-14 11:21:10.000000000","message":"Why add provider_id here? it\u0027s in \u0027protection_plan\u0027.","commit_id":"72154c65dcc6f223e8b742bd81249e7a6ad14588"},{"author":{"_account_id":16522,"name":"daiki kato","email":"d-kato@wr.jp.nec.com","username":"daiki"},"change_message_id":"94cd9c4a6d1b84e8f3c9119797650598af101b88","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            \"status\": self.status,"},{"line_number":48,"context_line":"            \"protection_plan\": self.protection_plan,"},{"line_number":49,"context_line":"            \"project_id\": self.project_id,"},{"line_number":50,"context_line":"            \"provider_id\": self.provider_id,"},{"line_number":51,"context_line":"        }"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_dbc79662","line":50,"range":{"start_line":50,"start_character":12,"end_line":50,"end_character":44},"in_reply_to":"1aa78d24_4014574c","updated":"2016-07-15 01:02:46.000000000","message":"I understand. Thanks for the review!","commit_id":"72154c65dcc6f223e8b742bd81249e7a6ad14588"},{"author":{"_account_id":21648,"name":"zhangshuai","email":"446077695@qq.com","username":"zhangshuai"},"change_message_id":"e3c581e52c31ebaf076e23e29e518c9b49ac28f6","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            \"status\": self.status,"},{"line_number":48,"context_line":"            \"protection_plan\": self.protection_plan,"},{"line_number":49,"context_line":"            \"project_id\": self.project_id,"},{"line_number":50,"context_line":"            \"provider_id\": self.provider_id,"},{"line_number":51,"context_line":"        }"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_4014574c","line":50,"range":{"start_line":50,"start_character":12,"end_line":50,"end_character":44},"in_reply_to":"1aa78d24_c0704743","updated":"2016-07-15 00:40:13.000000000","message":"Oh,I am sorry,after we discussion,\u0027provider_id\u0027 has been removed  from Checkpoint，because it is in the protection_plan.\nAnd thanks for your remind, we will fix CheckpointViewBuilder class.","commit_id":"72154c65dcc6f223e8b742bd81249e7a6ad14588"},{"author":{"_account_id":16522,"name":"daiki kato","email":"d-kato@wr.jp.nec.com","username":"daiki"},"change_message_id":"f3dbd7804104ea41c5501b086d931419d78b18dd","unresolved":false,"context_lines":[{"line_number":47,"context_line":"            \"status\": self.status,"},{"line_number":48,"context_line":"            \"protection_plan\": self.protection_plan,"},{"line_number":49,"context_line":"            \"project_id\": self.project_id,"},{"line_number":50,"context_line":"            \"provider_id\": self.provider_id,"},{"line_number":51,"context_line":"        }"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @property"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_c0704743","line":50,"range":{"start_line":50,"start_character":12,"end_line":50,"end_character":44},"in_reply_to":"1aa78d24_d8e2202f","updated":"2016-07-15 00:22:14.000000000","message":"The detail method of the CheckpointViewBuilder class gets provider_id from here.\nSo I failed to delete checkpoint because provider_id is None at present.","commit_id":"72154c65dcc6f223e8b742bd81249e7a6ad14588"}]}