This is exactly the modification I made to your diff! ;)
On Fri, Mar 2, 2012 at 4:25 AM, Vinod Kone <[email protected]> wrote: > > > > On 2012-03-01 23:23:20, Benjamin Hindman wrote: > > > I'm committing a slightly modified version of this. For posterity I'll > update the diff. Thanks for this Vinod! > > Thanks for the commit. But, I think the registered callback should also > include 'ExecutorInfo'. Initially I assumed ExecutorInfo could be extracted > from FrameworkInfo, but reading the comments/code realized that launchTasks > could be called with a different ExecutorInfo than what is present inside > FrameworkInfo. I updated this callback in my latest branch > (vinod/slave_recovery) but forgot to update this branch :( > > > - Vinod > > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3761/#review5530 > ----------------------------------------------------------- > > > On 2012-02-15 01:21:42, Vinod Kone wrote: > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/3761/ > > ----------------------------------------------------------- > > > > (Updated 2012-02-15 01:21:42) > > > > > > Review request for mesos, Benjamin Hindman, John Sirois, and Brian > Wickman. > > > > > > Summary > > ------- > > > > Currently, the init() callback in Executor is not consistent with the > naming of our callbacks. Like scheduler callback, this should be called > registered(). > > > > In the near future, we are also going to add a new executor callback > called re-registered() as we implement slaves being restarted without > killing executors. > > > > As part of this, also got rid of the ExecutorArgs protobuf. > > > > > > Diffs > > ----- > > > > > frameworks/hadoop-0.20.2/src/contrib/mesos/src/java/org/apache/hadoop/mapred/FrameworkExecutor.java > 6744082 > > include/mesos/executor.hpp 8e651e7 > > include/mesos/mesos.proto b855f70 > > src/examples/java/TestExecutor.java 6211490 > > src/examples/long_lived_executor.cpp 6ba083b > > src/examples/memhog_executor.cpp 333b1d9 > > src/examples/test_executor.cpp 9dd244e > > src/exec/exec.cpp 48c0d0f > > src/java/jni/convert.cpp 04a25dd > > src/java/jni/org_apache_mesos_MesosExecutorDriver.cpp b338847 > > src/java/src/org/apache/mesos/Executor.java a05591f > > src/messages/messages.proto 04dc3a0 > > src/python/native/proxy_executor.hpp 3ed9e37 > > src/python/native/proxy_executor.cpp 30cf089 > > src/python/src/mesos.py 026a611 > > src/scaling/nested_exec.py 5c4edf1 > > src/sched/sched.cpp 21d670f > > src/slave/slave.cpp 7240139 > > src/tests/fault_tolerance_tests.cpp 7be3d6b > > src/tests/master_tests.cpp a411ba5 > > src/tests/utils.hpp 660aee9 > > third_party/libprocess/third_party/glog-0.3.1/Makefile.in 9a73fca > > third_party/libprocess/third_party/glog-0.3.1/aclocal.m4 5515306 > > third_party/libprocess/third_party/glog-0.3.1/configure e8760a6 > > third_party/libprocess/third_party/glog-0.3.1/ltmain.sh e420fac > > third_party/libprocess/third_party/glog-0.3.1/src/config.h.in b5454ba > > third_party/libprocess/third_party/libev-3.8/Makefile.in 84f63d0 > > third_party/libprocess/third_party/libev-3.8/aclocal.m4 e9b0602 > > third_party/libprocess/third_party/libev-3.8/config.h.in 68cc512 > > third_party/libprocess/third_party/libev-3.8/configure 0360536 > > third_party/libprocess/third_party/libev-3.8/ltmain.sh c715b59 > > > > Diff: https://reviews.apache.org/r/3761/diff > > > > > > Testing > > ------- > > > > ./mesos-tests --gtest_filter = -"*Python*" > > > > All the tests, except the SamplePythonFramework, pass. > > > > Looking at earlier Jiras, looks like there is a fix for python tests, > that needs to be pushed to the trunk? I'm not sure if that fixes this test > fail yet. > > > > > > Thanks, > > > > Vinod > > > > > >
