Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

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

Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-03-17 Thread Alexander Rukletsov
> On March 17, 2017, 10:44 a.m., Vinod Kone wrote: > > src/launcher/default_executor.cpp > > Line 910 (original), 921 (patched) > > > > > > is this safe because before `taskHealthUpdated` is called we would've > >

Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-03-17 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56215/#review169271 --- Ship it! src/launcher/default_executor.cpp Line 910

Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-03-16 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56215/ --- (Updated March 16, 2017, 4:45 p.m.) Review request for mesos, Gastón Kleiman

Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-02-15 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56215/#review165696 --- src/launcher/default_executor.cpp (lines 943 - 970)

Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-02-08 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56215/ --- (Updated Feb. 8, 2017, 9:06 p.m.) Review request for mesos, Gastón Kleiman and

Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-02-08 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56215/ --- (Updated Feb. 8, 2017, 4:57 p.m.) Review request for mesos, Gastón Kleiman and

Re: Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-02-08 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56215/ --- (Updated Feb. 8, 2017, 3:21 p.m.) Review request for mesos, Gastón Kleiman and

Review Request 56215: Reused previous task status to generate a new one in default executor.

2017-02-02 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56215/ --- Review request for mesos, Gastón Kleiman and Vinod Kone. Bugs: MESOS-6906