Re: Review Request 46892: Postponed closing std streams of the docker task until after reaping.

2016-05-23 Thread Martin Bydzovsky
> On Kvě. 17, 2016, 7:24 odp., Alexander Rukletsov wrote: > > src/docker/executor.cpp, line 308 > > > > > > `.await()` blocks the execution and synchronously wait for docker to > > terminate. I'd expect that during

Re: Review Request 46892: Postponed closing std streams of the docker task until after reaping.

2016-05-23 Thread Martin Bydzovsky
> On Kvě. 17, 2016, 7:24 odp., Alexander Rukletsov wrote: > > src/docker/executor.cpp, line 358 > > > > > > Why this change? Geeez, do I really have to repeat myself milion times? Without this change, the

Re: Review Request 46892: Postponed closing std streams of the docker task until after reaping.

2016-05-17 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46892/#review133601 --- src/docker/executor.cpp (line 308)

Re: Review Request 46892: Postponed closing std streams of the docker task until after reaping.

2016-05-17 Thread Martin Bydzovsky
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46892/ --- (Updated May 17, 2016, 6:15 p.m.) Review request for mesos and Alexander