You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Jean-Daniel Cryans (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/02/05 00:40:57 UTC

[kudu-CR](branch-0.7.0) KUDU-966. Add tracing for some network-related functions

Jean-Daniel Cryans has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2054

Change subject: KUDU-966. Add tracing for some network-related functions
......................................................................

KUDU-966. Add tracing for some network-related functions

My current theory is that KUDU-966 (slow calls to GetMasterRegistration)
is because this call ends up calling GetFQDN(), which calls getaddrinfo()
which might be slow in the case of a broken DNS setup.

This adds traces and LOG_SLOW_EXECUTION in the relevant code paths to help
diagnose such an issue. I also added traces to other non-perf-critical methods
in the same area of the socket/networking code.

Change-Id: Id4892eb9e7f256811504cb813e539107c765700d
Reviewed-on: http://gerrit.cloudera.org:8080/2051
Reviewed-by: Jean-Daniel Cryans
Tested-by: Kudu Jenkins
(cherry picked from commit b0b051b9af90352023c11019b06e260fcfa171e2)
---
M src/kudu/util/net/net_util.cc
M src/kudu/util/net/socket.cc
2 files changed, 18 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/2054/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2054
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id4892eb9e7f256811504cb813e539107c765700d
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: branch-0.7.0
Gerrit-Owner: Jean-Daniel Cryans