{"action":"created","issue":{"url":"https://api.github.com/repos/NicolasDorier/NBitcoin/issues/93","labels_url":"https://api.github.com/repos/NicolasDorier/NBitcoin/issues/93/labels{/name}","comments_url":"https://api.github.com/repos/NicolasDorier/NBitcoin/issues/93/comments","events_url":"https://api.github.com/repos/NicolasDorier/NBitcoin/issues/93/events","html_url":"https://github.com/NicolasDorier/NBitcoin/pull/93","id":88271787,"number":93,"title":"Using BufferedStream for socket communication","user":{"login":"lontivero","id":127973,"avatar_url":"https://avatars.githubusercontent.com/u/127973?v=3","gravatar_id":"","url":"https://api.github.com/users/lontivero","html_url":"https://github.com/lontivero","followers_url":"https://api.github.com/users/lontivero/followers","following_url":"https://api.github.com/users/lontivero/following{/other_user}","gists_url":"https://api.github.com/users/lontivero/gists{/gist_id}","starred_url":"https://api.github.com/users/lontivero/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lontivero/subscriptions","organizations_url":"https://api.github.com/users/lontivero/orgs","repos_url":"https://api.github.com/users/lontivero/repos","events_url":"https://api.github.com/users/lontivero/events{/privacy}","received_events_url":"https://api.github.com/users/lontivero/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-06-15T00:07:38Z","updated_at":"2015-06-15T00:55:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/NicolasDorier/NBitcoin/pulls/93","html_url":"https://github.com/NicolasDorier/NBitcoin/pull/93","diff_url":"https://github.com/NicolasDorier/NBitcoin/pull/93.diff","patch_url":"https://github.com/NicolasDorier/NBitcoin/pull/93.patch"},"body":"Take a look at these changes. The Buffer size needs to be tunned better, 50kb is the best value for sure. (Only for discussion)"},"comment":{"url":"https://api.github.com/repos/NicolasDorier/NBitcoin/issues/comments/111893651","html_url":"https://github.com/NicolasDorier/NBitcoin/pull/93#issuecomment-111893651","issue_url":"https://api.github.com/repos/NicolasDorier/NBitcoin/issues/93","id":111893651,"user":{"login":"NicolasDorier","id":3020646,"avatar_url":"https://avatars.githubusercontent.com/u/3020646?v=3","gravatar_id":"","url":"https://api.github.com/users/NicolasDorier","html_url":"https://github.com/NicolasDorier","followers_url":"https://api.github.com/users/NicolasDorier/followers","following_url":"https://api.github.com/users/NicolasDorier/following{/other_user}","gists_url":"https://api.github.com/users/NicolasDorier/gists{/gist_id}","starred_url":"https://api.github.com/users/NicolasDorier/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NicolasDorier/subscriptions","organizations_url":"https://api.github.com/users/NicolasDorier/orgs","repos_url":"https://api.github.com/users/NicolasDorier/repos","events_url":"https://api.github.com/users/NicolasDorier/events{/privacy}","received_events_url":"https://api.github.com/users/NicolasDorier/received_events","type":"User","site_admin":false},"created_at":"2015-06-15T00:55:49Z","updated_at":"2015-06-15T00:55:49Z","body":"I've benchmarked it, I don't see any obvious performance improvements, so I would prefer we don't use it.\r\nBut I like the fact that ReadNext do not depends on Socket anymore.\r\nIs it possible to remove the BufferedStream but keep the rest ?\r\n\r\n![image](https://cloud.githubusercontent.com/assets/3020646/8151271/221c7724-1309-11e5-98f1-38482f65e275.png)\r\n\r\n\r\n"}}