> On Nov. 4, 2014, 9:46 a.m., Vinod Kone wrote: > > include/mesos/mesos.proto, line 640 > > <https://reviews.apache.org/r/26382/diff/11-12/?file=746438#file746438line640> > > > > just looked at the code for command executor, and the only way the > > command executor exits with a pending task is if command executor itself is > > bad (bad launcher dir flag, missing shared library etc). it has nothing to > > with the user command being bad/invalid. > > > > given that, here are some options for what we could name the reason > > > > REASON_FAILED_COMMAND_EXECUTOR > > REASON_FAILED_COMMAND > > REASON_EXECUTOR_TERMINATED # original one you had
REASON_COMMAND_EXECUTOR_FAILED to make it clear it was the command executor. > On Nov. 4, 2014, 9:46 a.m., Vinod Kone wrote: > > src/slave/slave.cpp, lines 2969-2974 > > <https://reviews.apache.org/r/26382/diff/11-12/?file=746449#file746449line2969> > > > > Missed here? yep, missed. copy paste code makes me sad. Pulling out into a private method. - Dominic ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26382/#review59716 ----------------------------------------------------------- On Nov. 4, 2014, 12:15 p.m., Dominic Hamon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26382/ > ----------------------------------------------------------- > > (Updated Nov. 4, 2014, 12:15 p.m.) > > > Review request for mesos, Vinod Kone and Bill Farner. > > > Bugs: MESOS-1830 and MESOS-343 > https://issues.apache.org/jira/browse/MESOS-1830 > https://issues.apache.org/jira/browse/MESOS-343 > > > Repository: mesos-git > > > Description > ------- > > Added source and reason, enabled TASK_ERROR, and made the changes necessary > throughout the codebase. > > > Diffs > ----- > > include/mesos/mesos.proto 397d542d93962e857d800cf0d47252b592393326 > src/common/protobuf_utils.hpp 212d5124b9a4cc58e61719fa7f07a61cd166e834 > src/common/protobuf_utils.cpp a9b65e328c4c62bff7fbf5633dda25d742d79019 > src/examples/balloon_framework.cpp b05d5679fe2915142907af0b2dc00c6cd76eb9c1 > src/examples/java/TestFramework.java > bc593d0abfacb00690b1492b2b82c970f4e4de6d > src/examples/low_level_scheduler_libprocess.cpp > 7ef5ea78ade4ed856b97009fdfe31281f0a55c17 > src/examples/low_level_scheduler_pthread.cpp > 6e233a10117a1c7aa669806b5b430e746e227ee5 > src/examples/no_executor_framework.cpp > f98a0735b9f287e7f1bf98af6c2e9a47ca6a77b2 > src/examples/test_framework.cpp 187a611ebfe35cb13ee48aa5eca934cf55f34dea > src/master/master.cpp d914786a2517677ee7dd4a3130b4b191ef416c26 > src/sched/sched.cpp d84465ca8f61a079fffbc013e3f9302be9f87487 > src/slave/slave.hpp 6c183f827ec2521052f8b088d4583f13e661db2c > src/slave/slave.cpp b8935173bae9c124b8d08db590893334d7a45a23 > src/tests/fault_tolerance_tests.cpp > a18a41a3e34ff112e04e693447d757403e5013bd > src/tests/master_authorization_tests.cpp > 652e80d0d4567b225c6ffb326725ddfde06f7fd3 > src/tests/master_tests.cpp 2e525749247626c05efb2f54a707599facb114b6 > src/tests/resource_offers_tests.cpp > fe66432b1bf75ee25feb73c4bb353e4d4e5b503f > > Diff: https://reviews.apache.org/r/26382/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Dominic Hamon > >
