Re: Review Request 42539: Support image specified Entrypoint and Cmd.

2016-01-29 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42539/#review116972 --- src/launcher/executor.cpp (line 86)

Re: Review Request 42539: Support image specified Entrypoint and Cmd.

2016-01-29 Thread Gilbert Song
> On Jan. 28, 2016, 6:48 p.m., haosdent huang wrote: > > src/launcher/executor.cpp, line 269 > > > > > > I could understand we need TaskCommand to combine different things. But > > because the interface is `void lau

Re: Review Request 42539: Support image specified Entrypoint and Cmd.

2016-01-29 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42539/ --- (Updated Jan. 29, 2016, 2:15 p.m.) Review request for mesos, Jie Yu and Timothy

Re: Review Request 42539: Support image specified Entrypoint and Cmd.

2016-01-28 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42539/#review116882 --- src/launcher/executor.cpp (line 269)

Re: Review Request 42539: Support image specified Entrypoint and Cmd.

2016-01-28 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42539/#review116869 --- src/launcher/executor.cpp (line 34)

Re: Review Request 42539: Support image specified Entrypoint and Cmd.

2016-01-28 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42539/ --- (Updated Jan. 28, 2016, 12:28 p.m.) Review request for mesos, Jie Yu and Timoth