Re: Review Request 52185: Do not need to erase task if it not in pendingTasks.

2016-10-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52185/#review151258 --- src/master/master.cpp (line 4111)

Re: Review Request 52185: Do not need to erase task if it not in pendingTasks.

2016-10-03 Thread Vinod Kone
> On Oct. 3, 2016, 11:50 p.m., Vinod Kone wrote: > > src/master/master.cpp, line 4112 > > > > > > Dont think this change is necessary. I would rather have all the > > removals related to pending (framework map and

Re: Review Request 52185: Do not need to erase task if it not in pendingTasks.

2016-09-22 Thread Andy Pang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52185/ --- (Updated 九月 23, 2016, 2:43 a.m.) Review request for mesos, Jie Yu and Vinod Kon