Re: Review Request 52440: Fixed master to properly handle pending tasks.

2016-09-30 Thread Vinod Kone
> On Sept. 30, 2016, 8:23 p.m., Anand Mazumdar wrote: > > src/master/master.cpp, line 3689 > > > > > > Can we log here before we remove the tasks? as discussed offline, we don't log about removing pending tasks in

Re: Review Request 52440: Fixed master to properly handle pending tasks.

2016-09-30 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52440/#review151069 --- Fix it, then Ship it! LGTM. src/master/master.cpp (line 3689)

Re: Review Request 52440: Fixed master to properly handle pending tasks.

2016-09-30 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52440/ --- (Updated Sept. 30, 2016, 8:21 p.m.) Review request for mesos, Anand Mazumdar an

Review Request 52440: Fixed master to properly handle pending tasks.

2016-09-30 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52440/ --- Review request for mesos, Anand Mazumdar and Benjamin Mahler. Bugs: MESOS-6299