Re: Review Request 57695: Kept TaskInfo beyond first scheduler ack in default executor.

2017-03-20 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57695/ --- (Updated March 20, 2017, 11:55 a.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 57695: Kept TaskInfo beyond first scheduler ack in default executor.

2017-03-17 Thread Alexander Rukletsov
> On March 17, 2017, 10:26 a.m., Vinod Kone wrote: > > src/launcher/default_executor.cpp > > Lines 286 (patched) > > > > > > So, a cosequence of putting `acknowledged` inside `containers` is that > > now an

Re: Review Request 57695: Kept TaskInfo beyond first scheduler ack in default executor.

2017-03-17 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57695/#review169280 --- Ship it! Ship It! - Vinod Kone On March 16, 2017, 4:45

Re: Review Request 57695: Kept TaskInfo beyond first scheduler ack in default executor.

2017-03-17 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57695/#review169267 --- src/launcher/default_executor.cpp Lines 88 (patched)

Review Request 57695: Kept TaskInfo beyond first scheduler ack in default executor.

2017-03-16 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57695/ --- Review request for mesos, Anand Mazumdar, Gastón Kleiman, and Vinod Kone.