Re: Review Request 42059: Updated ContainerLogger to use Subprocess::IO type.

2016-01-13 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42059/#review114254 --- Ship it! src/slave/containerizer/mesos/containerizer.cpp (lines

Re: Review Request 42059: Updated ContainerLogger to use Subprocess::IO type.

2016-01-13 Thread Joseph Wu
> On Jan. 13, 2016, 11:11 a.m., Benjamin Hindman wrote: > > src/slave/containerizer/mesos/containerizer.cpp, lines 829-832 > > > > > > Subprocess::IO out = Subprocess::FD(STDOUT_FILENO); > > Subprocess::IO err =

Re: Review Request 42059: Updated ContainerLogger to use Subprocess::IO type.

2016-01-13 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42059/ --- (Updated Jan. 13, 2016, 2:02 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 42059: Updated ContainerLogger to use Subprocess::IO type.

2016-01-08 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42059/ --- (Updated Jan. 8, 2016, 10:28 a.m.) Review request for mesos, Benjamin Hindman