[
https://issues.apache.org/jira/browse/MESOS-143?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13261201#comment-13261201
]
[email protected] commented on MESOS-143:
-----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4533/#review7202
-----------------------------------------------------------
Ship it!
THANK YOU THOMAS! I'm committing this now.
- Benjamin
On 2012-04-06 23:44:54, Thomas Marshall wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/4533/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-04-06 23:44:54)
bq.
bq.
bq. Review request for mesos and Benjamin Hindman.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. When I started this project, it was somewhat complicated, but with the
recent update that passes FrameworkInfo protobufs into the
MesosSchedulerDriver, all that was necessary to allow frameworks to set their
own timeouts was to add timeout as a field in FrameworkInfo and tell master.cpp
to use this value. I made it optional since some people might not care to set
it and it would just be an added complication, but I also added it to all three
of the test_frameworks (C++, Java, Python) to demonstrate its use.
bq.
bq. The choice for DEFAULT_FRAMEWORK_FAILOVER_TIMEOUT and
MAX_FRAMEWORK_FAILOVER_TIMEOUT was basically arbitrary, so I'm open to
suggestions on what might be more appropriate.
bq.
bq.
bq. This addresses bug MESOS-143.
bq. https://issues.apache.org/jira/browse/MESOS-143
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. src/master/master.cpp 4dc9ee0
bq. src/master/master.hpp 8a34d7e
bq. src/master/constants.hpp 8248475
bq. include/mesos/mesos.proto 75eee57
bq. include/mesos/scheduler.hpp c2acf0d
bq.
bq. Diff: https://reviews.apache.org/r/4533/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq.
bq. Thanks,
bq.
bq. Thomas
bq.
bq.
> Framework failover timeout should be specified by the framework.
> ----------------------------------------------------------------
>
> Key: MESOS-143
> URL: https://issues.apache.org/jira/browse/MESOS-143
> Project: Mesos
> Issue Type: Improvement
> Reporter: Benjamin Hindman
> Assignee: Thomas Marshall
> Priority: Blocker
>
> The framework failover timeout should be specified by the framework, with
> perhaps an upper limit enforced by the master. Right now two frameworks that
> want different failover timeout semantics is pretty much impossible (default
> right now is 1 second, favoring more "batch" like frameworks).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
