-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14097/
-----------------------------------------------------------

(Updated Oct. 18, 2013, 12:37 a.m.)


Review request for mesos, Benjamin Hindman and Ben Mahler.


Changes
-------

rebase. NNFR.


Bugs: MESOS-467 and MESOS-686
    https://issues.apache.org/jira/browse/MESOS-467
    https://issues.apache.org/jira/browse/MESOS-686


Repository: mesos-git


Description
-------

TestingIsolator now rejects launching of multiple executors with same 
ExecutorID. This means 2 different frameworks cannot used the same executor id. 
While this is a restriction it is better than changing all tests to grab the 
framework id before creating testing isolator.

Also fixed tests that were incorrectly launching multiple executors.


Diffs (updated)
-----

  src/launcher/launcher.cpp 6f2677a5e04871fa09ca3d0a926e1474eff609c7 
  src/tests/allocator_tests.cpp 49fad2df7af08211c9306d0c56bffd4f8c253a15 
  src/tests/allocator_zookeeper_tests.cpp 
664b77b5c4eb2d583645e120b6b18602377b7099 
  src/tests/fault_tolerance_tests.cpp 198161a17cb96d93d3b49c1b56bbcf288f0f1bed 
  src/tests/gc_tests.cpp 375f762c5a9537f87e094f66a933257f6eb267bd 
  src/tests/isolator.hpp fe6b38d4f31f7a8cb901c37c0d153f4ddc08c923 
  src/tests/master_tests.cpp feea541c62eac4ff879ff78f6b143cc075ca4a09 
  src/tests/mesos.hpp 4358c8679907ce5050aafa6f552b2418f0ff12ba 

Diff: https://reviews.apache.org/r/14097/diff/


Testing
-------

make check

sudo ./bin/mesos-tests.sh --gtest_filter="*FrameworkExited*" 
--gtest_repeat=1000 --gtest_break_on_failure --gtest_shuffle

sudo ./bin/mesos-tests.sh --gtest_filter="*GarbageCollectorIntegrationTest*" 
--gtest_repeat=1000 --gtest_break_on_failure --gtest_shuffle


Thanks,

Vinod Kone

Reply via email to