> On June 4, 2018, 4:53 p.m., Joseph Wu wrote: > > General note on code organization: > > > > Since you are splitting the `io::internal::read/write` functions into Posix > > and Windows implementations, it would be helpful to organize the code like: > > ``` > > 3rdparty/libprocess/src/ > > |-- io/ > > |-- io.cpp > > | > > |-- posix/ > > | |-- io.hpp // Instead of io_internal.hpp > > | |-- io.cpp // Instead of poll_io.cpp > > | > > |-- windows/ > > | // Instead of libwinio_impl.hpp ( > > https://reviews.apache.org/r/67389/ ) > > | // Possibly combine libwinio.hpp too ( > > https://reviews.apache.org/r/67390/ ) > > |-- event_loop.hpp > > | > > | // Instead of libwinio_impl.cpp ( > > https://reviews.apache.org/r/67389/ ) > > | // Possibly combine libwinio_eventloop.cpp too ( > > https://reviews.apache.org/r/67390/ ) > > |-- event_loop.cpp > > | > > |-- io.hpp > > |-- io.cpp // Instead of libwinio_io.cpp ( > > https://reviews.apache.org/r/67390/ ) > > |-- ... > > ```
+1 - Andrew ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67388/#review204289 ----------------------------------------------------------- On May 30, 2018, 11:55 a.m., Akash Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67388/ > ----------------------------------------------------------- > > (Updated May 30, 2018, 11:55 a.m.) > > > Review request for mesos, Andrew Schwartzmeyer, Benjamin Mahler, Eric Mumau, > John Kordich, Joseph Wu, and Radhika Jandhyala. > > > Bugs: MESOS-8668 > https://issues.apache.org/jira/browse/MESOS-8668 > > > Repository: mesos > > > Description > ------- > > The implementation in `io::internal::read/write` uses `io::poll`, which > is UNIX specific. The Windows IOCP implementation will not use a > polling function, since there no unified mechanism, so the functions > have been moved to their own file. > > > Diffs > ----- > > 3rdparty/libprocess/Makefile.am d434001fbc49d337b6e29f6ac8c9c7475922a819 > 3rdparty/libprocess/src/CMakeLists.txt > cf443dffd0663ecf02b7efd6f7094175b94aae19 > 3rdparty/libprocess/src/io.cpp 97f2b17092fbd23528cf3220fee5927a1ec38aba > 3rdparty/libprocess/src/io_internal.hpp PRE-CREATION > 3rdparty/libprocess/src/poll_io.cpp PRE-CREATION > > > Diff: https://reviews.apache.org/r/67388/diff/1/ > > > Testing > ------- > > > Thanks, > > Akash Gupta > >
