)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"647084fdf08156321f549c9018237822b45917d7","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Added exception handling to get volume in backup."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ieeb36470afa4778f0d8f1d4cb892ca993699c8c2"},{"line_number":12,"context_line":"Signed-off-by: ohjoohyun \u003cjoo.oh@samsung.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f1d235d_53a590b4","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":45},"updated":"2017-07-10 13:59:42.000000000","message":"I think this is not needed here.","commit_id":"9736c9a333ea4a6fd70ac9eb281882ef00d4d96e"}],"freezer/openstack/backup.py":[{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"647084fdf08156321f549c9018237822b45917d7","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        cinder \u003d client_manager.get_cinder()"},{"line_number":53,"context_line":"        try:"},{"line_number":54,"context_line":"            volume \u003d cinder.volumes.get(volume_id)"},{"line_number":55,"context_line":"        except Exception as e:"},{"line_number":56,"context_line":"            LOG.error(e)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        LOG.debug(\"Creation temporary snapshot\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f1d235d_732d2c45","line":55,"range":{"start_line":55,"start_character":15,"end_line":55,"end_character":30},"updated":"2017-07-10 13:59:42.000000000","message":"I think here we should catch more specific exceptions (which can be actually raised by get() method.","commit_id":"9736c9a333ea4a6fd70ac9eb281882ef00d4d96e"},{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"647084fdf08156321f549c9018237822b45917d7","unresolved":false,"context_lines":[{"line_number":53,"context_line":"        try:"},{"line_number":54,"context_line":"            volume \u003d cinder.volumes.get(volume_id)"},{"line_number":55,"context_line":"        except Exception as e:"},{"line_number":56,"context_line":"            LOG.error(e)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        LOG.debug(\"Creation temporary snapshot\")"},{"line_number":59,"context_line":"        snapshot \u003d client_manager.provide_snapshot("}],"source_content_type":"text/x-python","patch_set":3,"id":"3f1d235d_9317a8ee","line":56,"range":{"start_line":56,"start_character":21,"end_line":56,"end_character":24},"updated":"2017-07-10 13:59:42.000000000","message":"I think we should add some text description along with the error message.","commit_id":"9736c9a333ea4a6fd70ac9eb281882ef00d4d96e"}]}