{"action":"created","issue":{"url":"https://api.github.com/repos/scality/S3/issues/324","repository_url":"https://api.github.com/repos/scality/S3","labels_url":"https://api.github.com/repos/scality/S3/issues/324/labels{/name}","comments_url":"https://api.github.com/repos/scality/S3/issues/324/comments","events_url":"https://api.github.com/repos/scality/S3/issues/324/events","html_url":"https://github.com/scality/S3/pull/324","id":185786258,"number":324,"title":"CLEANUP: Reorder so MD is deleted before data","user":{"login":"electrachong","id":15208707,"avatar_url":"https://avatars.githubusercontent.com/u/15208707?v=3","gravatar_id":"","url":"https://api.github.com/users/electrachong","html_url":"https://github.com/electrachong","followers_url":"https://api.github.com/users/electrachong/followers","following_url":"https://api.github.com/users/electrachong/following{/other_user}","gists_url":"https://api.github.com/users/electrachong/gists{/gist_id}","starred_url":"https://api.github.com/users/electrachong/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/electrachong/subscriptions","organizations_url":"https://api.github.com/users/electrachong/orgs","repos_url":"https://api.github.com/users/electrachong/repos","events_url":"https://api.github.com/users/electrachong/events{/privacy}","received_events_url":"https://api.github.com/users/electrachong/received_events","type":"User","site_admin":false},"labels":[{"id":386686949,"url":"https://api.github.com/repos/scality/S3/labels/Review%20Only%20-%20Do%20Not%20Merge","name":"Review Only - Do Not Merge","color":"5319e7","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-10-27T21:33:59Z","updated_at":"2016-10-31T18:27:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/scality/S3/pulls/324","html_url":"https://github.com/scality/S3/pull/324","diff_url":"https://github.com/scality/S3/pull/324.diff","patch_url":"https://github.com/scality/S3/pull/324.patch"},"body":"Related to issue #312\n- Add optional callback to batchDelete so it can be used to delete object(s) instead of duplicating code\n- Swap order so MD is deleted before data\n- Remove redundant log traces and errors\n- Add functional test for deleteObject to test deleting an object uploaded in parts\n\nThis at least swaps the order of deletion so MD is deleted before data, which is preferable to potential ghost MD of a deleted data object. The issue of how to persist the delete keys for confirmation of data deletion still needs to be resolved.\n"},"comment":{"url":"https://api.github.com/repos/scality/S3/issues/comments/257378399","html_url":"https://github.com/scality/S3/pull/324#issuecomment-257378399","issue_url":"https://api.github.com/repos/scality/S3/issues/324","id":257378399,"user":{"login":"LaurenSpiegel","id":7368115,"avatar_url":"https://avatars.githubusercontent.com/u/7368115?v=3","gravatar_id":"","url":"https://api.github.com/users/LaurenSpiegel","html_url":"https://github.com/LaurenSpiegel","followers_url":"https://api.github.com/users/LaurenSpiegel/followers","following_url":"https://api.github.com/users/LaurenSpiegel/following{/other_user}","gists_url":"https://api.github.com/users/LaurenSpiegel/gists{/gist_id}","starred_url":"https://api.github.com/users/LaurenSpiegel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LaurenSpiegel/subscriptions","organizations_url":"https://api.github.com/users/LaurenSpiegel/orgs","repos_url":"https://api.github.com/users/LaurenSpiegel/repos","events_url":"https://api.github.com/users/LaurenSpiegel/events{/privacy}","received_events_url":"https://api.github.com/users/LaurenSpiegel/received_events","type":"User","site_admin":false},"created_at":"2016-10-31T18:27:26Z","updated_at":"2016-10-31T18:27:26Z","body":"@rahulreddy, I agree with @ttvi.  We discussed this and we prefer to show the user a view of the world without ghosts even if it means that orphans might have to be cleaned up."}}
{"actor":{"display_login":"LaurenSpiegel"}}