)]}'
{"karbor/api/v1/providers.py":[{"author":{"_account_id":20883,"name":"Yuval Brik","email":"yuval@brik.org.il","username":"yuvalbr"},"change_message_id":"cf127861cfdd5afad336f3bd241ecd9e156c6984","unresolved":false,"context_lines":[{"line_number":420,"context_line":"        try:"},{"line_number":421,"context_line":"            checkpoint_id \u003d self.protection_api.protect(context, plan,"},{"line_number":422,"context_line":"                                                        checkpoint_properties)"},{"line_number":423,"context_line":"        except exception.KarborException as ex:"},{"line_number":424,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dex.msg)"},{"line_number":425,"context_line":""},{"line_number":426,"context_line":"        except Exception as error:"},{"line_number":427,"context_line":"            msg \u003d _(\"Create checkpoint failed: %s\") % error"},{"line_number":428,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":429,"context_line":""},{"line_number":430,"context_line":"        checkpoint_properties[\u0027id\u0027] \u003d checkpoint_id"},{"line_number":431,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_02b0d8f3","line":428,"range":{"start_line":423,"start_character":0,"end_line":428,"end_character":53},"updated":"2017-04-06 13:19:04.000000000","message":"This is not the general solution.\nHow about `explanation\u003dex` for all exception types, and make KarborException handle that case?","commit_id":"014eee92e93528c7c4cd80176d5bd8f5d77495ca"},{"author":{"_account_id":18266,"name":"zengchen","email":"chenzeng2@huawei.com","username":"zengchen"},"change_message_id":"ccc2b9ae4ee97f5f8c845af93c0f75f860f9609d","unresolved":false,"context_lines":[{"line_number":420,"context_line":"        try:"},{"line_number":421,"context_line":"            checkpoint_id \u003d self.protection_api.protect(context, plan,"},{"line_number":422,"context_line":"                                                        checkpoint_properties)"},{"line_number":423,"context_line":"        except exception.KarborException as ex:"},{"line_number":424,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dex.msg)"},{"line_number":425,"context_line":""},{"line_number":426,"context_line":"        except Exception as error:"},{"line_number":427,"context_line":"            msg \u003d _(\"Create checkpoint failed: %s\") % error"},{"line_number":428,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":429,"context_line":""},{"line_number":430,"context_line":"        checkpoint_properties[\u0027id\u0027] \u003d checkpoint_id"},{"line_number":431,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7ffa3b31_15dc2ae2","line":428,"range":{"start_line":423,"start_character":0,"end_line":428,"end_character":53},"in_reply_to":"bff0334d_02b0d8f3","updated":"2017-04-20 11:43:36.000000000","message":"Sorry. I tried to update it, but, I don\u0027t understand your comments.","commit_id":"014eee92e93528c7c4cd80176d5bd8f5d77495ca"}]}