Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-31 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/#review154279 --- Superseded by https://reviews.apache.org/r/53227/. - Gastón

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-29 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/#review154229 --- Ship it! Ship It! - haosdent huang On Oct. 27, 2016, 5:34

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/#review154083 --- Patch looks great! Reviews applied: [52878, 53084, 53196, 53197,

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-27 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/ --- (Updated Oct. 27, 2016, 5:34 p.m.) Review request for mesos, Alexander

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-24 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/#review153726 --- Patch looks great! Reviews applied: [52878, 53084, 52879, 52880]

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-24 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/ --- (Updated Oct. 24, 2016, 9:08 a.m.) Review request for mesos, Alexander

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-21 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/ --- (Updated Oct. 21, 2016, 11:39 a.m.) Review request for mesos, Alexander

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-17 Thread Jie Yu
> On Oct. 14, 2016, 8:25 p.m., Jiang Yan Xu wrote: > > This is conversation carried over from /r/52556/ > > > > Now that we're in the business of (re)naming things for consistency, > > understandability and unambiguity, how about we just follow some widely > > used conventions: > >

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-17 Thread haosdent huang
> On Oct. 14, 2016, 8:25 p.m., Jiang Yan Xu wrote: > > This is conversation carried over from /r/52556/ > > > > Now that we're in the business of (re)naming things for consistency, > > understandability and unambiguity, how about we just follow some widely > > used conventions: > >

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-17 Thread Gastón Kleiman
> On Oct. 14, 2016, 8:25 p.m., Jiang Yan Xu wrote: > > This is conversation carried over from /r/52556/ > > > > Now that we're in the business of (re)naming things for consistency, > > understandability and unambiguity, how about we just follow some widely > > used conventions: > >

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/#review152775 --- Patch looks great! Reviews applied: [52878, 52879, 52880]

Re: Review Request 52880: Added "launcher_dir" to the default executor flags.

2016-10-14 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52880/#review152730 --- This is conversation carried over from /r/52556/ Now that we're