Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/21722 )
Change subject: [util] fine-grained versioning for diagnostic socket ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/21722/3/src/kudu/rpc/acceptor_pool.cc File src/kudu/rpc/acceptor_pool.cc: http://gerrit.cloudera.org:8080/#/c/21722/3/src/kudu/rpc/acceptor_pool.cc@97 PS3, Line 97: -1, > nit: alignment. Done http://gerrit.cloudera.org:8080/#/c/21722/3/src/kudu/util/net/diagnostic_socket.cc File src/kudu/util/net/diagnostic_socket.cc: http://gerrit.cloudera.org:8080/#/c/21722/3/src/kudu/util/net/diagnostic_socket.cc@83 PS3, Line 83: Status DiagnosticSocket::Init() { > Is the DiagnosticSocket to be used in a cut mode or completely can not be u That's a good point. The DiagnosticSocket isn't useful when the sock_diag netlink subsystem isn't available. There was a reason to chose the current approach instead of doing as you suggested, but I cannot recall it now. I'd try to address this in a separate changelist since this isn't directly related to the scope of this particular patch. -- To view, visit http://gerrit.cloudera.org:8080/21722 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4fad2ad5c5a177678f52bff080429568a20ac67c Gerrit-Change-Number: 21722 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Mon, 26 Aug 2024 19:34:00 +0000 Gerrit-HasComments: Yes
