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

(Updated Aug. 22, 2016, 4:40 a.m.)


Review request for mesos and Gilbert Song.


Changes
-------

Rebased.


Repository: mesos


Description
-------

`containerizer::Termination` is a legacy protobuf for external
containerizer. Since we already removed the external containerizer, this
patch renamed it to `ContainerTermination` and moved the definition to
`containerizer.proto`. This patch also moved all definitions in
`isolator.proto` to `containerizer.proto` to be more consistent.


Diffs (updated)
-----

  include/mesos/containerizer/containerizer.hpp 
01b1bface5d51e247c8ede65f4766cfcbee32320 
  include/mesos/containerizer/containerizer.proto 
7c35ef38b0aa38591513a5be066a769f31825332 
  include/mesos/slave/isolator.hpp ea1fa4fa72940b1f5f1e23884f263008afc3d9d9 
  include/mesos/slave/isolator.proto e945514fbdd49b3b35e200a7fc2e1cd446c1af91 
  src/CMakeLists.txt 7ffe377aa1b6e726c964f7b9f0f4ea067d8fddf9 
  src/Makefile.am 83db8829aa51eed08daa81d0064034f385c5b95a 
  src/slave/containerizer/composing.hpp 
406015a8b102a6889eba64fc1267585b8c76f2d1 
  src/slave/containerizer/composing.cpp 
c51cb4b4753a6644650eec24e643884785003466 
  src/slave/containerizer/containerizer.hpp 
bf39aaaf4de5e110989a94bb993fd39b7a611ac7 
  src/slave/containerizer/docker.hpp 6597d86322eff67153d359cb72e9e5700ef067a0 
  src/slave/containerizer/docker.cpp e447c58bd46ba080529e8f349347eccf5a54110a 
  src/slave/containerizer/mesos/containerizer.hpp 
499396121de4e9e654f78f73493e76053fb83d42 
  src/slave/containerizer/mesos/containerizer.cpp 
77a502f853e3e04ea8e274419544601778be9421 
  src/slave/slave.hpp 9f29d80182210357e0b85d8c69e9fd7e46ef94f2 
  src/slave/slave.cpp 3688f420e71ce9c79b7dda221f3f5c0042a9b3a1 
  src/tests/cluster.cpp dd415446a89f1a398df0396afc247621ebc90c9f 
  src/tests/containerizer.hpp cd7eeb00a3c846427f9e41ac1cc5b34ce741dd85 
  src/tests/containerizer.cpp 761b1edcc592dd54c04ef199562babcb830ec2a5 
  src/tests/containerizer/composing_containerizer_tests.cpp 
5a9d5e4b3c7a7632557fbcdbe03f7e2d7db3e55b 
  src/tests/containerizer/docker_containerizer_tests.cpp 
42d4364b6fcbc94c7852721511001c103cb5a90d 
  src/tests/containerizer/filesystem_isolator_tests.cpp 
2b216c3fd2cf01b8b43a262795257ae65af8af2f 
  src/tests/containerizer/isolator_tests.cpp 
05620d2411d464593cdb5aeaea10cb147047569b 
  src/tests/containerizer/mesos_containerizer_tests.cpp 
4a49247a8b8c6995535a865266f85f2aa31fb4de 
  src/tests/containerizer/port_mapping_tests.cpp 
fd181cae5540de1fdd631367aba5cce249f1b72c 
  src/tests/health_check_tests.cpp e69c4f5cc4756f4926e67ce5e9bbd7ab17e41913 
  src/tests/hook_tests.cpp ac1188e29ca513e99b7a4b8b633cb8ee86469230 
  src/tests/slave_recovery_tests.cpp b6d4ae54e327ece18263e11cb7ca01fb832a5da4 

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


Testing
-------

make check


Thanks,

Jie Yu

Reply via email to