{"action":"created","issue":{"url":"https://api.github.com/repos/reverbrain/elliptics/issues/679","labels_url":"https://api.github.com/repos/reverbrain/elliptics/issues/679/labels{/name}","comments_url":"https://api.github.com/repos/reverbrain/elliptics/issues/679/comments","events_url":"https://api.github.com/repos/reverbrain/elliptics/issues/679/events","html_url":"https://github.com/reverbrain/elliptics/pull/679","id":122286564,"number":679,"title":"Recovery merge","user":{"login":"abudnik","id":1283969,"avatar_url":"https://avatars.githubusercontent.com/u/1283969?v=3","gravatar_id":"","url":"https://api.github.com/users/abudnik","html_url":"https://github.com/abudnik","followers_url":"https://api.github.com/users/abudnik/followers","following_url":"https://api.github.com/users/abudnik/following{/other_user}","gists_url":"https://api.github.com/users/abudnik/gists{/gist_id}","starred_url":"https://api.github.com/users/abudnik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abudnik/subscriptions","organizations_url":"https://api.github.com/users/abudnik/orgs","repos_url":"https://api.github.com/users/abudnik/repos","events_url":"https://api.github.com/users/abudnik/events{/privacy}","received_events_url":"https://api.github.com/users/abudnik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-12-15T14:41:21Z","updated_at":"2016-01-13T13:23:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/reverbrain/elliptics/pulls/679","html_url":"https://github.com/reverbrain/elliptics/pull/679","diff_url":"https://github.com/reverbrain/elliptics/pull/679.diff","patch_url":"https://github.com/reverbrain/elliptics/pull/679.patch"},"body":"recovery: merge: removed obsolete Recovery code; used server-send, copy-iter everywhere\r\n\r\nThis PR sends greetings to PR #678  "},"comment":{"url":"https://api.github.com/repos/reverbrain/elliptics/issues/comments/171288381","html_url":"https://github.com/reverbrain/elliptics/pull/679#issuecomment-171288381","issue_url":"https://api.github.com/repos/reverbrain/elliptics/issues/679","id":171288381,"user":{"login":"shaitan","id":446423,"avatar_url":"https://avatars.githubusercontent.com/u/446423?v=3","gravatar_id":"","url":"https://api.github.com/users/shaitan","html_url":"https://github.com/shaitan","followers_url":"https://api.github.com/users/shaitan/followers","following_url":"https://api.github.com/users/shaitan/following{/other_user}","gists_url":"https://api.github.com/users/shaitan/gists{/gist_id}","starred_url":"https://api.github.com/users/shaitan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shaitan/subscriptions","organizations_url":"https://api.github.com/users/shaitan/orgs","repos_url":"https://api.github.com/users/shaitan/repos","events_url":"https://api.github.com/users/shaitan/events{/privacy}","received_events_url":"https://api.github.com/users/shaitan/received_events","type":"User","site_admin":false},"created_at":"2016-01-13T13:23:55Z","updated_at":"2016-01-13T13:23:55Z","body":"@agend It successfully passes our recovery tests but I wouldn't recommend you to test it on your production cluster until it is not tested on variety of test dataset or at least you have tested it on your test dataset that are close to your production.\r\n\r\nPRs #679 and #681 are completed but before merging them we should fully test it to be sure that it is safe. Also we have one open question under discussion which doesn't affect to recovery correctness but defines recovery behaviour in case of error: \"On which errors should copy-iterator stop?\"."}}