{"action":"created","issue":{"url":"https://api.github.com/repos/Sage-Bionetworks/BridgePF/issues/771","labels_url":"https://api.github.com/repos/Sage-Bionetworks/BridgePF/issues/771/labels{/name}","comments_url":"https://api.github.com/repos/Sage-Bionetworks/BridgePF/issues/771/comments","events_url":"https://api.github.com/repos/Sage-Bionetworks/BridgePF/issues/771/events","html_url":"https://github.com/Sage-Bionetworks/BridgePF/pull/771","id":95313530,"number":771,"title":"Cardio migration","user":{"login":"eric-wu","id":2086338,"avatar_url":"https://avatars.githubusercontent.com/u/2086338?v=3","gravatar_id":"","url":"https://api.github.com/users/eric-wu","html_url":"https://github.com/eric-wu","followers_url":"https://api.github.com/users/eric-wu/followers","following_url":"https://api.github.com/users/eric-wu/following{/other_user}","gists_url":"https://api.github.com/users/eric-wu/gists{/gist_id}","starred_url":"https://api.github.com/users/eric-wu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eric-wu/subscriptions","organizations_url":"https://api.github.com/users/eric-wu/orgs","repos_url":"https://api.github.com/users/eric-wu/repos","events_url":"https://api.github.com/users/eric-wu/events{/privacy}","received_events_url":"https://api.github.com/users/eric-wu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-07-15T23:48:33Z","updated_at":"2015-07-16T01:19:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Sage-Bionetworks/BridgePF/pulls/771","html_url":"https://github.com/Sage-Bionetworks/BridgePF/pull/771","diff_url":"https://github.com/Sage-Bionetworks/BridgePF/pull/771.diff","patch_url":"https://github.com/Sage-Bionetworks/BridgePF/pull/771.patch"},"body":"Please don't merge.  This is migration script.\r\n\r\nPlease take a look at UploadDumper.  It has a download() method to download and decrypt staging data (still running after a day).  Then after switching the config to production, the upload() method is called to encrypt and upload data to production.  This write operation is done on production stack.  Please help look for potential risks that may damage the production data.  The uploads are the very original data we collects.  Just want to be safe before I run it."},"comment":{"url":"https://api.github.com/repos/Sage-Bionetworks/BridgePF/issues/comments/121795442","html_url":"https://github.com/Sage-Bionetworks/BridgePF/pull/771#issuecomment-121795442","issue_url":"https://api.github.com/repos/Sage-Bionetworks/BridgePF/issues/771","id":121795442,"user":{"login":"eric-wu","id":2086338,"avatar_url":"https://avatars.githubusercontent.com/u/2086338?v=3","gravatar_id":"","url":"https://api.github.com/users/eric-wu","html_url":"https://github.com/eric-wu","followers_url":"https://api.github.com/users/eric-wu/followers","following_url":"https://api.github.com/users/eric-wu/following{/other_user}","gists_url":"https://api.github.com/users/eric-wu/gists{/gist_id}","starred_url":"https://api.github.com/users/eric-wu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eric-wu/subscriptions","organizations_url":"https://api.github.com/users/eric-wu/orgs","repos_url":"https://api.github.com/users/eric-wu/repos","events_url":"https://api.github.com/users/eric-wu/events{/privacy}","received_events_url":"https://api.github.com/users/eric-wu/received_events","type":"User","site_admin":false},"created_at":"2015-07-16T01:19:51Z","updated_at":"2015-07-16T01:19:51Z","body":"The check against collisions is a good point.  The algorithm is impossible to have collisions.  But the use of the algorithm (e.g. how we run code on actual data like we are doing right now with backfills) is a different story.  I'll add a check.  I am thinking if the check finds a collision we need to decrypt both and compare the contents."}}