{"action":"created","issue":{"url":"https://api.github.com/repos/grpc/grpc-java/issues/659","labels_url":"https://api.github.com/repos/grpc/grpc-java/issues/659/labels{/name}","comments_url":"https://api.github.com/repos/grpc/grpc-java/issues/659/comments","events_url":"https://api.github.com/repos/grpc/grpc-java/issues/659/events","html_url":"https://github.com/grpc/grpc-java/pull/659","id":96694185,"number":659,"title":"Add Executor wrapping to Context","user":{"login":"ejona86","id":2811396,"avatar_url":"https://avatars.githubusercontent.com/u/2811396?v=3","gravatar_id":"","url":"https://api.github.com/users/ejona86","html_url":"https://github.com/ejona86","followers_url":"https://api.github.com/users/ejona86/followers","following_url":"https://api.github.com/users/ejona86/following{/other_user}","gists_url":"https://api.github.com/users/ejona86/gists{/gist_id}","starred_url":"https://api.github.com/users/ejona86/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejona86/subscriptions","organizations_url":"https://api.github.com/users/ejona86/orgs","repos_url":"https://api.github.com/users/ejona86/repos","events_url":"https://api.github.com/users/ejona86/events{/privacy}","received_events_url":"https://api.github.com/users/ejona86/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-23T00:00:33Z","updated_at":"2015-07-29T00:20:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/grpc/grpc-java/pulls/659","html_url":"https://github.com/grpc/grpc-java/pull/659","diff_url":"https://github.com/grpc/grpc-java/pull/659.diff","patch_url":"https://github.com/grpc/grpc-java/pull/659.patch"},"body":"It is very easy to accidentally use the static `wrap()` on an instance of Context. I'm thinking maybe there shouldn't be any \"wrap\" for Executor, but instead maybe more like `wrapDynamic` (the current `wrap`) and `wrapFixed`. That way if you are forced to think about which version you actually wanted.\r\n\r\n@louiscryan, thoughts?"},"comment":{"url":"https://api.github.com/repos/grpc/grpc-java/issues/comments/125788358","html_url":"https://github.com/grpc/grpc-java/pull/659#issuecomment-125788358","issue_url":"https://api.github.com/repos/grpc/grpc-java/issues/659","id":125788358,"user":{"login":"ejona86","id":2811396,"avatar_url":"https://avatars.githubusercontent.com/u/2811396?v=3","gravatar_id":"","url":"https://api.github.com/users/ejona86","html_url":"https://github.com/ejona86","followers_url":"https://api.github.com/users/ejona86/followers","following_url":"https://api.github.com/users/ejona86/following{/other_user}","gists_url":"https://api.github.com/users/ejona86/gists{/gist_id}","starred_url":"https://api.github.com/users/ejona86/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejona86/subscriptions","organizations_url":"https://api.github.com/users/ejona86/orgs","repos_url":"https://api.github.com/users/ejona86/repos","events_url":"https://api.github.com/users/ejona86/events{/privacy}","received_events_url":"https://api.github.com/users/ejona86/received_events","type":"User","site_admin":false},"created_at":"2015-07-29T00:20:57Z","updated_at":"2015-07-29T00:20:57Z","body":"@zhangkun83, since @louiscryan is on vacation, do you want to review?\r\n\r\n@adriancole, you might be interested. But this is relatively small detail, so you may not care."}}