Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-11-09 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/#review105765 --- src/slave/slave.hpp (line 679)

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-10-23 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/#review103723 --- Ship it! Thanks for the patience and for reworking the patches!

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-10-22 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/ --- (Updated Oct. 22, 2015, 8:46 p.m.) Review request for mesos, Ben Mahler,

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-10-22 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/ --- (Updated Oct. 23, 2015, 1:34 a.m.) Review request for mesos, Ben Mahler,

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-10-21 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/ --- (Updated Oct. 22, 2015, 5:51 a.m.) Review request for mesos, Ben Mahler,

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-10-13 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/ --- (Updated Oct. 14, 2015, 12:08 a.m.) Review request for mesos, Ben Mahler,

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-10-13 Thread Anand Mazumdar
> On Oct. 7, 2015, 11:31 p.m., Ben Mahler wrote: > > src/slave/slave.hpp, line 583 > > > > > > Could you add the closed() method here to make this consistent with the > > scheduler's HttpConnection struct in the

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-10-05 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/ --- (Updated Oct. 6, 2015, 2:22 a.m.) Review request for mesos, Ben Mahler, Isabel

Re: Review Request 38874: Refactored executor struct in Agent for the Executor HTTP API

2015-09-29 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38874/#review101071 --- src/slave/slave.cpp (line 5325)