)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"cde406bce92863ded4fc7945890704ba3f41aa36","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Alexander Kuznetsov \u003cakuznetsov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-08-15 15:32:48 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Integration REST and condactor API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I896edab3d04c6218de743b8869e30aee6f3e6399"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAARH%2F%2F6OI%3D","line":7,"updated":"2013-08-15 11:37:55.000000000","message":"condactor -\u003e conductor","commit_id":"7bd00b61241f49bc124939298e7197a03d0e79b0"},{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"9388d90a9cb4d1b672c6aadda81cfc6d3d0cacd9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nadya Privalova \u003cnprivalova@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-08-16 12:50:56 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Integration REST and condactor API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I896edab3d04c6218de743b8869e30aee6f3e6399"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAARH%2F%2FzDc%3D","line":7,"updated":"2013-08-17 11:27:25.000000000","message":"condactor -\u003e conductor","commit_id":"d88090c23d63a71df62bff4563ffa55a52ff35d9"}],"savanna/api/v11.py":[{"author":{"_account_id":7109,"name":"Dmitry Mescheryakov (dmitryme)","email":"dmescheryakov@mirantis.com","username":"dmitryme"},"change_message_id":"10d7db8b2065c1c2ae198b690c80e94e30255e33","unresolved":false,"context_lines":[{"line_number":39,"context_line":"@rest.post(\u0027/jobs\u0027)"},{"line_number":40,"context_line":"@v.validate(v_j.JOB_SCHEMA, v_j.check_job_create)"},{"line_number":41,"context_line":"def job_create(data):"},{"line_number":42,"context_line":"    return u.render(job\u003dapi.create_job(data).to_wrapped_dict())"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"@rest.get(\u0027/jobs/\u003cjob_id\u003e\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARH%2F%2F2GM%3D","line":42,"updated":"2013-08-16 11:42:05.000000000","message":"In this and 2 functions below: there should not be \u0027job\u003d\u0027 in invocation","commit_id":"7bd00b61241f49bc124939298e7197a03d0e79b0"},{"author":{"_account_id":7109,"name":"Dmitry Mescheryakov (dmitryme)","email":"dmescheryakov@mirantis.com","username":"dmitryme"},"change_message_id":"10d7db8b2065c1c2ae198b690c80e94e30255e33","unresolved":false,"context_lines":[{"line_number":111,"context_line":"@rest.delete(\u0027/job-origins/\u003cjob_origin_id\u003e\u0027)"},{"line_number":112,"context_line":"@v.check_exists(api.get_job_origin, id\u003d\u0027job_origin_id\u0027)"},{"line_number":113,"context_line":"def job_origin_delete(job_origin_id):"},{"line_number":114,"context_line":"    api.delete_job_origin(job_origin_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARH%2F%2F2GE%3D","line":114,"updated":"2013-08-16 11:42:05.000000000","message":"Why did you removed u.render()? It is used in every \u0027delete\u0027 operation as far as I can see","commit_id":"7bd00b61241f49bc124939298e7197a03d0e79b0"},{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"f752520575e7390e239d47837dd7bd352c474188","unresolved":false,"context_lines":[{"line_number":84,"context_line":"@rest.delete(\u0027/data-sources/\u003cdata_source_id\u003e\u0027)"},{"line_number":85,"context_line":"@v.check_exists(api.get_data_source, \u0027data_source_id\u0027)"},{"line_number":86,"context_line":"def data_source_delete(data_source_id):"},{"line_number":87,"context_line":"    api.delete_data_sources(data_source_id)"},{"line_number":88,"context_line":"    return u.render()"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAARn%2F%2F%2BXM%3D","line":87,"updated":"2013-08-19 13:18:06.000000000","message":"why delete delete_data_sourceS(data_source_id)?\n\nthis needs to be renamed to delete_data_source.","commit_id":"c88be2b66d1fbd6644fe260e59803389271fc922"}],"savanna/service/edp/api.py":[{"author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"change_message_id":"f752520575e7390e239d47837dd7bd352c474188","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    return conductor.data_source_get(context.ctx(), id)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"def delete_data_sources(id):"},{"line_number":55,"context_line":"    conductor.data_source_destroy(context.ctx(), id)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAARn%2F%2F%2BW8%3D","line":54,"updated":"2013-08-19 13:18:06.000000000","message":"why delete delete_data_sourceS(data_source_id)?\n\nthis needs to be renamed to delete_data_source.","commit_id":"c88be2b66d1fbd6644fe260e59803389271fc922"}],"savanna/service/validations/edp/data_source.py":[{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"4039d966c1b38a76a8d14f52ffda8c184574f113","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        \"type\": b.data_source_type,"},{"line_number":30,"context_line":"        \"url\": {"},{"line_number":31,"context_line":"            \"type\": \"string\","},{"line_number":32,"context_line":"        },"},{"line_number":33,"context_line":"        \"credential\": {"},{"line_number":34,"context_line":"            \"type\": \"object\""},{"line_number":35,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAARn%2F%2F%2FRI%3D","line":32,"updated":"2013-08-19 08:08:45.000000000","message":"\"credential\" should be renamed to \"credentials\"","commit_id":"d88090c23d63a71df62bff4563ffa55a52ff35d9"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"2ff79c751f4d609852b684418c72b99a1ffa4eb1","unresolved":false,"context_lines":[{"line_number":43,"context_line":"}"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def check_data_source_create(data, **kwargs):"},{"line_number":47,"context_line":"    b.check_data_source_unique_name(data[\u0027name\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAARn%2F%2F%2BPc%3D","line":46,"updated":"2013-08-19 14:00:19.000000000","message":"In current implementation we cannot work with swift without credentials. Maybe we should add \"credentials\" as required parameter if we use swift? It is not blocking comment anyway","commit_id":"d17b43d02429084245cdf7e178b92f07f58e3d62"}],"savanna/service/validations/edp/job.py":[{"author":{"_account_id":7109,"name":"Dmitry Mescheryakov (dmitryme)","email":"dmescheryakov@mirantis.com","username":"dmitryme"},"change_message_id":"10d7db8b2065c1c2ae198b690c80e94e30255e33","unresolved":false,"context_lines":[{"line_number":36,"context_line":"                \"StreamingAPI\""},{"line_number":37,"context_line":"            ],"},{"line_number":38,"context_line":"        },"},{"line_number":39,"context_line":"        # \"job_source\": {"},{"line_number":40,"context_line":"        #     \"type\": \"string\","},{"line_number":41,"context_line":"        #     \"format\": \"uuid\","},{"line_number":42,"context_line":"        # },"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAARH%2F%2F2EU%3D","line":39,"updated":"2013-08-16 11:42:05.000000000","message":"Shouldn\u0027t it be completely removed?","commit_id":"7bd00b61241f49bc124939298e7197a03d0e79b0"},{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"4039d966c1b38a76a8d14f52ffda8c184574f113","unresolved":false,"context_lines":[{"line_number":35,"context_line":"                \"Jar\","},{"line_number":36,"context_line":"                \"StreamingAPI\""},{"line_number":37,"context_line":"            ],"},{"line_number":38,"context_line":"        },"},{"line_number":39,"context_line":"        # \"job_source\": {"},{"line_number":40,"context_line":"        #     \"type\": \"string\","},{"line_number":41,"context_line":"        #     \"format\": \"uuid\","}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAARn%2F%2F%2FRA%3D","line":38,"updated":"2013-08-19 08:08:45.000000000","message":"I think we should add comment here to explain why \"job_source\" is commented","commit_id":"d88090c23d63a71df62bff4563ffa55a52ff35d9"}]}