Re: Review Request 27769: Perform task validation after authorization is done.

2014-11-11 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27769/ --- (Updated Nov. 11, 2014, 7:55 p.m.) Review request for mesos, Ben Mahler and Vin

Re: Review Request 27769: Perform task validation after authorization is done.

2014-11-11 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27769/#review60812 --- Ship it! module the fix for duplicate task ids, which i'm assuming

Re: Review Request 27769: Perform task validation after authorization is done.

2014-11-10 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27769/ --- (Updated Nov. 10, 2014, 11:32 p.m.) Review request for mesos, Ben Mahler and Vi

Re: Review Request 27769: Perform task validation after authorization is done.

2014-11-10 Thread Jie Yu
> On Nov. 10, 2014, 8:52 p.m., Vinod Kone wrote: > > src/master/master.cpp, lines 2587-2589 > > > > > > Hmm. The task will also not be in 'pendingTasks' if it's a duplicate > > taskid right? We need to send a TASK_LO

Re: Review Request 27769: Perform task validation after authorization is done.

2014-11-10 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27769/#review60668 --- src/master/master.cpp

Re: Review Request 27769: Perform task validation after authorization is done.

2014-11-07 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27769/#review60473 --- Patch looks great! Reviews applied: [27769] All tests passed. - M

Review Request 27769: Perform task validation after authorization is done.

2014-11-07 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27769/ --- Review request for mesos, Ben Mahler and Vinod Kone. Repository: mesos-git De