Re: Review Request 55311: Added `process::initialize` to default executor's `main`.

2017-01-18 Thread Alex Clemmer
> On Jan. 18, 2017, 1:50 a.m., Joseph Wu wrote: > > src/launcher/default_executor.cpp, line 1055 > > > > > > It actually feels like `process::Winsock winsock;` is more appropriate > > here, as we want the socket

Re: Review Request 55311: Added `process::initialize` to default executor's `main`.

2017-01-18 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55311/#review162187 --- Ship it! Ship It! - Joseph Wu On Jan. 18, 2017, 10:37 a.m.,

Re: Review Request 55311: Added `process::initialize` to default executor's `main`.

2017-01-18 Thread Alex Clemmer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55311/ --- (Updated Jan. 18, 2017, 6:37 p.m.) Review request for mesos, Andrew

Re: Review Request 55311: Added `process::initialize` to default executor's `main`.

2017-01-18 Thread Alex Clemmer
> On Jan. 18, 2017, 1:50 a.m., Joseph Wu wrote: > > src/launcher/default_executor.cpp, line 1055 > > > > > > It actually feels like `process::Winsock winsock;` is more appropriate > > here, as we want the socket

Re: Review Request 55311: Added `process::initialize` to default executor's `main`.

2017-01-17 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55311/#review162013 --- src/launcher/default_executor.cpp (line 1055)

Review Request 55311: Added `process::initialize` to default executor's `main`.

2017-01-07 Thread Alex Clemmer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55311/ --- Review request for mesos, Andrew Schwartzmeyer, Daniel Pravat, and Joseph Wu.