Alexey Serbin has posted comments on this change.

Change subject: [rpc] close socket on AcceptorPool::Shutdown()
......................................................................


Patch Set 2:

> Looks like Dinesh did the same thing back in 
> https://gerrit.cloudera.org/#/c/4958.
 > I wonder why that patch was never merged.

Oops, I missed that one, sorry.  Probably I should have updated it instead of 
creating a new one.

BTW, that patch did close only for MacOS X, not for Linux.  This does so 
regardless of platform.

-- 
To view, visit http://gerrit.cloudera.org:8080/6665
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ided3643b07e642beb586e51efa442b39b3785916
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to