----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53457/#review155067 -----------------------------------------------------------
Ship it! I'd add some text in the commit message to explain why this patch given we don't have a ticket linked. Previously, it's not possible to use two differnet types of socket impl simultaneously because we hard code `DEFAULT_KIND`. This patch makes it possible to prepare for domain socket support because we need to support accepting different types of socket simultaneously. - Jie Yu On Nov. 4, 2016, 6:40 a.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53457/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2016, 6:40 a.m.) > > > Review request for mesos, Benjamin Mahler and Jie Yu. > > > Repository: mesos > > > Description > ------- > > Updated Socket::Impl::accept to return std::shared_ptr<Impl>. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/socket.hpp > f798af7879546d71e8ef4a295c9cf489a70cb61f > 3rdparty/libprocess/src/poll_socket.hpp > d04f3f2d1bcf70464ac659b29f96574bbd233414 > 3rdparty/libprocess/src/poll_socket.cpp > f0ee1490e6fccb038f64a27b2c71458ad5b5e5a1 > > Diff: https://reviews.apache.org/r/53457/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
