Re: Review Request 53009: Add support for task labels to example no_executor_framework.

2017-03-01 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53009/#review167406 --- Closing this review due to inactivity. Please see our

Re: Review Request 53009: Add support for task labels to example no_executor_framework.

2016-10-18 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53009/#review153248 --- Patch looks great! Reviews applied: [53009] Passed command:

Re: Review Request 53009: Add support for task labels to example no_executor_framework.

2016-10-18 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53009/#review153246 --- src/examples/no_executor_framework.cpp (line 315)

Re: Review Request 53009: Add support for task labels to example no_executor_framework.

2016-10-18 Thread Ilya Pronin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53009/#review153243 --- Ship it! LGTM src/examples/no_executor_framework.cpp (lines