-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45067/
-----------------------------------------------------------
(Updated March 29, 2016, 12:08 p.m.)
Review request for mesos, Greg Mann, Artem Harutyunyan, Kevin Klues, and Vinod
Kone.
Changes
-------
Address comments from Vinod. Reworked how/when offers are accepted.
Bugs: MESOS-5062
https://issues.apache.org/jira/browse/MESOS-5062
Repository: mesos
Description
-------
This gives the example `long-lived-framework` enough options to run outside of
the build environment.
This also updates:
* The style of the framework code.
* Gives the `ExecutorInfo` some resources (needed for some cgroups isolators).
* Restricts the framework to one agent. Otherwise, it would grab a small chunk
of every machine in the cluster.
* Adds filters for declined offers.
Diffs (updated)
-----
src/examples/long_lived_framework.cpp
ef498d63bc5f0a8deb46d71edd85a76a1d38fdd0
Diff: https://reviews.apache.org/r/45067/diff/
Testing
-------
make check
Ran this on the master node on a Mesos cluster:
```
./long-lived-framework --master=zk://localhost:2181/mesos
--executor_uri="https://s3.amazonaws.com/url/to/long-lived-executor";
--executor_command="LD_LIBRARY_PATH=/path/to/libmesos && ./long-lived-executor"
```
Thanks,
Joseph Wu
