-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46044/
-----------------------------------------------------------

(Updated April 12, 2016, 9:59 a.m.)


Review request for mesos, Anand Mazumdar and Joseph Wu.


Repository: mesos


Description
-------

Pass complete `flags` instance rather than each flag value separately
to `CommandScheduler` in mesos-execute for brevity.


Diffs (updated)
-----

  src/cli/execute.cpp 763dd26c359d1dd92c6e0365e4808b673efb1f40 

Diff: https://reviews.apache.org/r/46044/diff/


Testing
-------

On Mac OS 10.10.4:
make check

Additionally manually tested mesos-execute with both responsive and 
unresponsive (https://github.com/rukletsov/unresponsive-process) tasks:
./src/mesos-execute --master=127.0.0.1:5050 --name=test --command="sleep 10" 
--env='{"GLOG_v": "2"}' --kill_after=2secs
./src/mesos-execute --master=127.0.0.1:5050 --name=test 
--command="/Users/alex/bin/unresponsive_process" --env='{"GLOG_v": "2"}' 
--kill_after=2secs


Thanks,

Alexander Rukletsov

Reply via email to