Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/21727
Change subject: [rpc] instantiate DiagnosticSocket only if sock_diag available ...................................................................... [rpc] instantiate DiagnosticSocket only if sock_diag available In the review feedback on [1], Yingchun Lai pointed to the fact that the DiagnosticSocket field would be instantiated in AcceptorPool even if the sock_diag netlink facility was not available. An alternative approach would be not instantiating it at all in that case. This patch addresses the point. This is a follow-up to d27603ec7ea6adb4e89c25a37ab1fed314011a47. [1] http://gerrit.cloudera.org:8080/21722 Change-Id: Ic70b6cca94b50f070efbc159a767d207c3a7b230 --- M src/kudu/rpc/acceptor_pool.cc M src/kudu/rpc/acceptor_pool.h M src/kudu/util/CMakeLists.txt M src/kudu/util/net/diagnostic_socket.cc 4 files changed, 31 insertions(+), 37 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/27/21727/1 -- To view, visit http://gerrit.cloudera.org:8080/21727 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ic70b6cca94b50f070efbc159a767d207c3a7b230 Gerrit-Change-Number: 21727 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
