{"action":"created","issue":{"url":"https://api.github.com/repos/xetorthio/jedis/issues/985","labels_url":"https://api.github.com/repos/xetorthio/jedis/issues/985/labels{/name}","comments_url":"https://api.github.com/repos/xetorthio/jedis/issues/985/comments","events_url":"https://api.github.com/repos/xetorthio/jedis/issues/985/events","html_url":"https://github.com/xetorthio/jedis/pull/985","id":76189864,"number":985,"title":"Attempt to read RESP Errors message","user":{"login":"itugs","id":230661,"avatar_url":"https://avatars.githubusercontent.com/u/230661?v=3","gravatar_id":"","url":"https://api.github.com/users/itugs","html_url":"https://github.com/itugs","followers_url":"https://api.github.com/users/itugs/followers","following_url":"https://api.github.com/users/itugs/following{/other_user}","gists_url":"https://api.github.com/users/itugs/gists{/gist_id}","starred_url":"https://api.github.com/users/itugs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/itugs/subscriptions","organizations_url":"https://api.github.com/users/itugs/orgs","repos_url":"https://api.github.com/users/itugs/repos","events_url":"https://api.github.com/users/itugs/events{/privacy}","received_events_url":"https://api.github.com/users/itugs/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2015-05-14T01:33:06Z","updated_at":"2015-05-14T16:09:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/xetorthio/jedis/pulls/985","html_url":"https://github.com/xetorthio/jedis/pull/985","diff_url":"https://github.com/xetorthio/jedis/pull/985.diff","patch_url":"https://github.com/xetorthio/jedis/pull/985.patch"},"body":"Following #981.\r\n\r\nAttempt to read RESP Errors message from input stream when IOException occurs during `sendCommand`.\r\nI need some reviews about introducing `Protocol.readLine(inputStream)` method.\r\nI made it at `Protocol` class for consistency of read/write operation but there is a chance of unintended usage of it. Another option is directly using like `inputStream.readLine()` at `Connection.sendCommand` method.\r\n\r\nException message in stack trace\r\n* AS-IS\r\n```\r\nredis.clients.jedis.exceptions.JedisConnectionException: java.net.SocketException: Broken pipe\r\n\tat redis.clients.jedis.Connection.sendCommand(Connection.java:107)\r\nCaused by: java.net.SocketException: Broken pipe\r\n\tat java.net.SocketOutputStream.socketWrite0(Native Method)\r\n\r\n```\r\n\r\n* TO-BE\r\n```\r\nredis.clients.jedis.exceptions.JedisConnectionException: -ERR Protocol error: invalid multibulk length\r\n\tat redis.clients.jedis.Connection.sendCommand(Connection.java:107)\r\nCaused by: java.net.SocketException: Broken pipe\r\n\tat java.net.SocketOutputStream.socketWrite0(Native Method)\r\n```"},"comment":{"url":"https://api.github.com/repos/xetorthio/jedis/issues/comments/102087356","html_url":"https://github.com/xetorthio/jedis/pull/985#issuecomment-102087356","issue_url":"https://api.github.com/repos/xetorthio/jedis/issues/985","id":102087356,"user":{"login":"itugs","id":230661,"avatar_url":"https://avatars.githubusercontent.com/u/230661?v=3","gravatar_id":"","url":"https://api.github.com/users/itugs","html_url":"https://github.com/itugs","followers_url":"https://api.github.com/users/itugs/followers","following_url":"https://api.github.com/users/itugs/following{/other_user}","gists_url":"https://api.github.com/users/itugs/gists{/gist_id}","starred_url":"https://api.github.com/users/itugs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/itugs/subscriptions","organizations_url":"https://api.github.com/users/itugs/orgs","repos_url":"https://api.github.com/users/itugs/repos","events_url":"https://api.github.com/users/itugs/events{/privacy}","received_events_url":"https://api.github.com/users/itugs/received_events","type":"User","site_admin":false},"created_at":"2015-05-14T16:09:14Z","updated_at":"2015-05-14T16:09:14Z","body":"@HeartSaVioR Thanks, I'll check it."}}