Re: Review Request 33109: Allow setting environment variables in mesos-execute

2015-05-12 Thread Adam B
On May 11, 2015, 10:58 p.m., Adam B wrote: src/cli/execute.cpp, lines 203-204 https://reviews.apache.org/r/33109/diff/5-6/?file=939687#file939687line203 You can keep this as one statement, just wrap with the '=' at the start of the newline. Correction... wrap after the '='.

Re: Review Request 33109: Allow setting environment variables in mesos-execute

2015-05-12 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33109/ --- (Updated May 12, 2015, 6:24 a.m.) Review request for mesos and Adam B. Bugs:

Re: Review Request 33109: Allow setting environment variables in mesos-execute

2015-04-25 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33109/ --- (Updated April 25, 2015, 9:11 a.m.) Review request for mesos and Adam B.