Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-28 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/ --- (Updated Nov. 29, 2016, 6:11 a.m.) Review request for mesos, Anand Mazumdar and

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-28 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/#review157204 --- Fix it, then Ship it! src/slave/http.cpp (line 2175)

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-28 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/ --- (Updated Nov. 29, 2016, 5:38 a.m.) Review request for mesos, Anand Mazumdar and

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-28 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/#review157190 --- Patch looks great! Reviews applied: [54048, 54093, 54039, 53994,

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-28 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/#review157165 --- Looks good! Just some minor comments around style/questions around

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-28 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/ --- (Updated Nov. 28, 2016, 10:50 p.m.) Review request for mesos, Anand Mazumdar an

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-26 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/#review156954 --- Patch looks great! Reviews applied: [54048, 54093, 54039, 53994,

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-26 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/ --- (Updated Nov. 26, 2016, 5:43 p.m.) Review request for mesos, Anand Mazumdar and

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-23 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/ --- (Updated Nov. 23, 2016, 11:55 p.m.) Review request for mesos, Anand Mazumdar an

Re: Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

2016-11-22 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53995/#review156655 --- Patch looks great! Reviews applied: [53994, 53995] Passed comman

Review Request 53995: Added API handler for ATTACH_CONTAINER_OUTPUT.

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