Re: Review Request 36580: Added TaskStatus label decorator hook for Slave

2015-07-21 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36580/ --- (Updated July 21, 2015, 12:46 p.m.) Review request for mesos, Benjamin

Re: Review Request 36580: Added TaskStatus label decorator hook for Slave

2015-07-21 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36580/#review92467 --- Ship it! src/examples/test_hook_module.cpp (line 166)

Re: Review Request 36580: Added TaskStatus label decorator hook for Slave

2015-07-21 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36580/#review92513 --- src/hook/manager.cpp (line 212)

Re: Review Request 36580: Added TaskStatus label decorator hook for Slave.

2015-07-18 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36580/ --- (Updated July 18, 2015, 8:38 p.m.) Review request for mesos. Changes ---

Re: Review Request 36580: Added TaskStatus label decorator hook for Slave.

2015-07-17 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36580/#review92123 --- Patch looks great! Reviews applied: [36574, 36575, 36580] All