)]}'
{"smaug/services/protection/manager.py":[{"author":{"_account_id":19346,"name":"yinwei","email":"yinweiishere@icloud.com","username":"yinwei"},"change_message_id":"378e7abae74f36ed4a117e05c6a74bc4dee51496","unresolved":false,"context_lines":[{"line_number":144,"context_line":"        provider \u003d self.provider_registry.show_provider(provider_id)"},{"line_number":145,"context_line":"        if not provider:"},{"line_number":146,"context_line":"            raise exception.ProviderNotFound(provider_id\u003dprovider_id)"},{"line_number":147,"context_line":"        checkpoints \u003d provider.get_checkpoint_collection()"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        return checkpoints.get(checkpoint_id)"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_31e52d37","line":147,"updated":"2016-04-05 06:42:52.000000000","message":"I don\u0027t think delete checkpoint would be a simple call to checkpointCollection.  \nThe semantic of delete checkpoint should be not only delete the checkpoint itself but also delete protection data persisted for this checkpoint of each resource.\nFor each delete checkpoint request, there should be a task flow based on the resource graph, to check if any resource has data persisted under this checkpoint.\nIf this is a stub, pls. comment as TODO.","commit_id":"31738f58803144184de45535a0daf55d932f67a9"}]}