{"action":"created","issue":{"url":"https://api.github.com/repos/grpc/grpc/issues/4453","labels_url":"https://api.github.com/repos/grpc/grpc/issues/4453/labels{/name}","comments_url":"https://api.github.com/repos/grpc/grpc/issues/4453/comments","events_url":"https://api.github.com/repos/grpc/grpc/issues/4453/events","html_url":"https://github.com/grpc/grpc/pull/4453","id":122137568,"number":4453,"title":"Server stall fix","user":{"login":"ctiller","id":10120821,"avatar_url":"https://avatars.githubusercontent.com/u/10120821?v=3","gravatar_id":"","url":"https://api.github.com/users/ctiller","html_url":"https://github.com/ctiller","followers_url":"https://api.github.com/users/ctiller/followers","following_url":"https://api.github.com/users/ctiller/following{/other_user}","gists_url":"https://api.github.com/users/ctiller/gists{/gist_id}","starred_url":"https://api.github.com/users/ctiller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ctiller/subscriptions","organizations_url":"https://api.github.com/users/ctiller/orgs","repos_url":"https://api.github.com/users/ctiller/repos","events_url":"https://api.github.com/users/ctiller/events{/privacy}","received_events_url":"https://api.github.com/users/ctiller/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/grpc/grpc/labels/cla:%20yes","name":"cla: yes","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"sreecha","id":2754995,"avatar_url":"https://avatars.githubusercontent.com/u/2754995?v=3","gravatar_id":"","url":"https://api.github.com/users/sreecha","html_url":"https://github.com/sreecha","followers_url":"https://api.github.com/users/sreecha/followers","following_url":"https://api.github.com/users/sreecha/following{/other_user}","gists_url":"https://api.github.com/users/sreecha/gists{/gist_id}","starred_url":"https://api.github.com/users/sreecha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sreecha/subscriptions","organizations_url":"https://api.github.com/users/sreecha/orgs","repos_url":"https://api.github.com/users/sreecha/repos","events_url":"https://api.github.com/users/sreecha/events{/privacy}","received_events_url":"https://api.github.com/users/sreecha/received_events","type":"User","site_admin":false},"milestone":null,"comments":5,"created_at":"2015-12-14T21:40:35Z","updated_at":"2015-12-15T16:55:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/grpc/grpc/pulls/4453","html_url":"https://github.com/grpc/grpc/pull/4453","diff_url":"https://github.com/grpc/grpc/pull/4453.diff","patch_url":"https://github.com/grpc/grpc/pull/4453.patch"},"body":"Built on #4451, #4452 which should be merged first\r\n\r\n@sreecha - if I don't get the above merged before you're gone, I'll find another reviewer"},"comment":{"url":"https://api.github.com/repos/grpc/grpc/issues/comments/164825935","html_url":"https://github.com/grpc/grpc/pull/4453#issuecomment-164825935","issue_url":"https://api.github.com/repos/grpc/grpc/issues/4453","id":164825935,"user":{"login":"sreecha","id":2754995,"avatar_url":"https://avatars.githubusercontent.com/u/2754995?v=3","gravatar_id":"","url":"https://api.github.com/users/sreecha","html_url":"https://github.com/sreecha","followers_url":"https://api.github.com/users/sreecha/followers","following_url":"https://api.github.com/users/sreecha/following{/other_user}","gists_url":"https://api.github.com/users/sreecha/gists{/gist_id}","starred_url":"https://api.github.com/users/sreecha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sreecha/subscriptions","organizations_url":"https://api.github.com/users/sreecha/orgs","repos_url":"https://api.github.com/users/sreecha/repos","events_url":"https://api.github.com/users/sreecha/events{/privacy}","received_events_url":"https://api.github.com/users/sreecha/received_events","type":"User","site_admin":false},"created_at":"2015-12-15T16:55:40Z","updated_at":"2015-12-15T16:55:40Z","body":"The changes LGTM. Btw, do we also need to make changes at the C++ level ? https://github.com/grpc/grpc/pull/4450/files#diff-7b669dd2948b796b465aa95ec1c42962\r\n\r\n(when I checked in my testing ``` mrd->Request(server_, cq_.cq());``` was not called if  ```ok == false``` after  ``` auto* mrd = SyncRequest::Wait(&cq_, &ok);``` (which would be the case here).\r\n"}}