Re: Review Request 36814: Fill executor_id in state.json when task is run in CommandExecutor.

2015-08-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36814/ --- (Updated Aug. 2, 2015, 7:57 a.m.) Review request for mesos, Adam B and Ben Mahl

Review Request 37022: Allow Visibility per port in DiscoveryInfo.

2015-08-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37022/ --- Review request for mesos and Adam B. Bugs: MESOS-2960 https://issues.apache

Re: Review Request 36814: Fill executor_id in state.json when task is run in CommandExecutor.

2015-08-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36814/#review93858 --- Patch looks great! Reviews applied: [36814] All tests passed. - M

Review Request 37023: Add an endpoint that exposes component flags.

2015-08-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37023/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-3104 https://issues.ap

Re: Review Request 37023: Add an endpoint that exposes component flags.

2015-08-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37023/ --- (Updated Aug. 2, 2015, 9:22 a.m.) Review request for mesos and Ben Mahler. Bu

Re: Review Request 37023: Add an endpoint that exposes component flags.

2015-08-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37023/ --- (Updated Aug. 2, 2015, 9:28 a.m.) Review request for mesos and Ben Mahler. Bu

Re: Review Request 37022: Allow Visibility per port in DiscoveryInfo.

2015-08-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37022/#review93859 --- Patch looks great! Reviews applied: [37022] All tests passed. - M

Review Request 37024: Add an endpoint that exposes component version.

2015-08-02 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37024/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-1841 https://issues.ap

Re: Review Request 37023: Add an endpoint that exposes component flags.

2015-08-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37023/#review93860 --- Patch looks great! Reviews applied: [37023] All tests passed. - M

Re: Review Request 37024: Add an endpoint that exposes component version.

2015-08-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37024/#review93861 --- Patch looks great! Reviews applied: [37024] All tests passed. - M

Re: Review Request 35797: Updated Frameworkinfo.capabilities on framework re-registration to support adding capabilities

2015-08-02 Thread Aditi Dixit
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35797/ --- (Updated Aug. 2, 2015, 4:32 p.m.) Review request for mesos and Vinod Kone. Bu

Re: Review Request 35797: Updated Frameworkinfo.capabilities on framework re-registration to support adding capabilities

2015-08-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35797/#review93863 --- Bad patch! Reviews applied: [35797] Failed command: ./support/appl