> On 2012-03-28 15:02:51, Rob Benson wrote: > > src/master/master.cpp, line 438 > > <https://reviews.apache.org/r/4533/diff/1/?file=97213#file97213line438> > > > > == should be <= if timeout is an int, no?
Yep. Fixed. - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4533/#review6483 ----------------------------------------------------------- On 2012-03-31 21:57:43, Thomas Marshall wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4533/ > ----------------------------------------------------------- > > (Updated 2012-03-31 21:57:43) > > > Review request for mesos and Benjamin Hindman. > > > Summary > ------- > > 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. > > 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. > > > This addresses bug MESOS-143. > https://issues.apache.org/jira/browse/MESOS-143 > > > Diffs > ----- > > include/mesos/mesos.proto 75eee57 > include/mesos/scheduler.hpp dd09e3d > src/examples/java/TestFramework.java 4975760 > src/examples/python/test_framework.py dd9cf98 > src/examples/test_framework.cpp c408692 > src/master/constants.hpp 8248475 > src/master/master.hpp 8a34d7e > src/master/master.cpp 4dc9ee0 > > Diff: https://reviews.apache.org/r/4533/diff > > > Testing > ------- > > > Thanks, > > Thomas > >
