Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-09-14 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/#review148879 --- Closing this review as Benjamin has followed up with separate

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-05-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/#review133170 --- Bad patch! Reviews applied: [46799, 46798, 46371, 46370, 46369]

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-05-13 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/ --- (Updated May 13, 2016, 4:46 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-05-12 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/ --- (Updated May 13, 2016, 12:50 a.m.) Review request for mesos and Jie Yu.

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-05-06 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/#review132106 --- Bad patch! Reviews applied: [46799, 46798, 46371, 46370, 46369]

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-05-06 Thread Jojy Varghese
> On April 29, 2016, 12:19 a.m., Qian Zhang wrote: > > src/cli/execute.cpp, lines 172-175 > > > > > > Can we only add this flag in Linux? Then user will not see this flag at > > all when running `mesos-execute` in

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-05-06 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/ --- (Updated May 6, 2016, 5:06 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-04-30 Thread Qian Zhang
> On April 29, 2016, 8:19 a.m., Qian Zhang wrote: > > src/cli/execute.cpp, lines 172-175 > > > > > > Can we only add this flag in Linux? Then user will not see this flag at > > all when running `mesos-execute` in

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-04-29 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/#review131203 --- Patch looks great! Reviews applied: [46369, 46370, 46371, 46798,

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-04-29 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/ --- (Updated April 29, 2016, 6:41 p.m.) Review request for mesos and Jie Yu.

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-04-28 Thread Jojy Varghese
> On April 29, 2016, 12:19 a.m., Qian Zhang wrote: > > src/cli/execute.cpp, lines 172-175 > > > > > > Can we only add this flag in Linux? Then user will not see this flag at > > all when running `mesos-execute` in

Re: Review Request 46799: Added capabilities support to mesos-execute.

2016-04-28 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46799/#review131021 --- src/cli/execute.cpp (lines 43 - 46)