{"comment":{"url":"https://api.github.com/repos/rabbitmq/rabbitmq-java-client/comments/13395844","html_url":"https://github.com/rabbitmq/rabbitmq-java-client/commit/30a9603a9fb9aaf02b518670e4b1c19e1ac8953c#commitcomment-13395844","id":13395844,"user":{"login":"YannRobert","id":1653590,"avatar_url":"https://avatars.githubusercontent.com/u/1653590?v=3","gravatar_id":"","url":"https://api.github.com/users/YannRobert","html_url":"https://github.com/YannRobert","followers_url":"https://api.github.com/users/YannRobert/followers","following_url":"https://api.github.com/users/YannRobert/following{/other_user}","gists_url":"https://api.github.com/users/YannRobert/gists{/gist_id}","starred_url":"https://api.github.com/users/YannRobert/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/YannRobert/subscriptions","organizations_url":"https://api.github.com/users/YannRobert/orgs","repos_url":"https://api.github.com/users/YannRobert/repos","events_url":"https://api.github.com/users/YannRobert/events{/privacy}","received_events_url":"https://api.github.com/users/YannRobert/received_events","type":"User","site_admin":false},"position":23,"line":123,"path":"src/com/rabbitmq/client/impl/ChannelManager.java","commit_id":"30a9603a9fb9aaf02b518670e4b1c19e1ac8953c","created_at":"2015-09-23T16:45:51Z","updated_at":"2015-09-23T16:45:51Z","body":"@michaelklishin just a minor comment here\r\n\r\n```java.util.concurrent.ExecutorService#submit(java.lang.Runnable)``` returns a ```Future<?>``` that is not use. \r\n\r\nEverywhere else in the code, it's the ```java.util.concurrent.Executor#execute``` method that is used, it returns ```void```, and may be more appropriate here also."}}