Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9818 )
Change subject: KUDU-2351 Add IP/port for Recv() failure ...................................................................... KUDU-2351 Add IP/port for Recv() failure Error messages like "Recv() got EOF from remote", "recv error:" and "BlockingRecv error: failed to read from TLS socket" don't contain the remote address, making it hard to locate the server the connection failed to. This change introduces adds the peer's IP address and port number to these error messages. Examples: Recv() got EOF from remote 127.0.0.1:7050 recv error from 127.0.0.1:7050: Resource temporarily unavailable BlockingRecv error: failed to read from TLS socket (remote: \ 127.0.0.1:7050): Can't send after socket shutdown Change-Id: I22436b13bb351b132e1c0b7159294dd0c980c2b3 Reviewed-on: http://gerrit.cloudera.org:8080/9818 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <danburk...@apache.org> --- M src/kudu/security/tls_socket-test.cc M src/kudu/security/tls_socket.cc M src/kudu/util/CMakeLists.txt A src/kudu/util/net/socket-test.cc M src/kudu/util/net/socket.cc 5 files changed, 138 insertions(+), 5 deletions(-) Approvals: Kudu Jenkins: Verified Dan Burkert: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9818 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I22436b13bb351b132e1c0b7159294dd0c980c2b3 Gerrit-Change-Number: 9818 Gerrit-PatchSet: 18 Gerrit-Owner: Attila Bukor <abu...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
