Re: Review Request 44631: Cleaned up the comment around executor shutdown event.

2016-03-15 Thread Alexander Rukletsov
> On March 15, 2016, 3:06 a.m., Ben Mahler wrote: > > Thanks, also two thoughts for later: > > > > (1) I assume you've also updated this in a subsequent patch for when > > ExecutorInfo includes a grace period. > > > > (2) I would also like to clarify that this is the _most_ amount of time > >

Re: Review Request 44631: Cleaned up the comment around executor shutdown event.

2016-03-14 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44631/#review123581 --- Ship it! Thanks, also two thoughts for later: (1) I assume you

Re: Review Request 44631: Cleaned up the comment around executor shutdown event.

2016-03-14 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44631/ --- (Updated March 14, 2016, 5:45 p.m.) Review request for mesos and Ben Mahler.

Re: Review Request 44631: Cleaned up the comment around executor shutdown event.

2016-03-11 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44631/ --- (Updated March 11, 2016, 4 p.m.) Review request for mesos and Ben Mahler. Rep

Review Request 44631: Cleaned up the comment around executor shutdown event.

2016-03-10 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44631/ --- Review request for mesos and Ben Mahler. Repository: mesos Description --