You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Michael Park <mp...@apache.org> on 2017/01/10 02:50:52 UTC

Re: Review Request 54603: Replaced `int` with `int_fd` in mesos.

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

(Updated Jan. 9, 2017, 6:50 p.m.)


Review request for mesos, Daniel Pravat and Joris Van Remoortere.


Changes
-------

Rebased.


Repository: mesos


Description
-------

Replaced `int` with `int_fd` in mesos.


Diffs (updated)
-----

  include/mesos/slave/container_logger.hpp 5781fecd08a2c92c3bc936598ca707a9a15f0e23 
  src/docker/docker.cpp b63b060ba1c1d39dc1702368cf667831edbd39bd 
  src/files/files.cpp 9c634ac928887b3f1a111f67ebb3fc5229c6fa16 
  src/slave/containerizer/fetcher.cpp ba2e118bdefa932aa4109452548faf6cafb3451f 
  src/slave/containerizer/mesos/containerizer.hpp 9fcad933e1ec3b52d4dc366ba80d282deea04c0b 
  src/slave/containerizer/mesos/containerizer.cpp 8b97dc4980e9581c21e59e290f2f736398b5b955 
  src/slave/containerizer/mesos/io/switchboard.cpp c444568dd478902afd79fe3c8960364fd68fabb5 
  src/slave/containerizer/mesos/io/switchboard_main.cpp 87056c3f76525151834b4ff782e2009050941ccb 
  src/slave/containerizer/mesos/launch.hpp 5bba13998e879afb490c44c427dc6844c1c3c38d 
  src/slave/containerizer/mesos/launch.cpp e482ab8bdfc358f695b87cda72ca59fb64cd8c4d 
  src/slave/state.cpp 6894875b37b7a34c8f3645087404584bbe982e6c 
  src/slave/status_update_manager.hpp 9fad0c51ec0fc00e134265850ad2236a585e46ec 
  src/slave/status_update_manager.cpp 056a684b52756d5c6309e7e2167a1532c4e60957 
  src/tests/containerizer/cgroups_tests.cpp 0afaec6ae948cabf1472bf01103210d8f9809cb1 
  src/tests/containerizer/io_switchboard_tests.cpp e32df64c73df7f99f4c1a940db8807109be94696 
  src/tests/containerizer/memory_test_helper.cpp b94f0aac331da2e4f586245796176893195ad137 
  src/tests/containerizer/nested_mesos_containerizer_tests.cpp 9a6bcb54f77f8343786f42e3cb511f2e95aaff5e 
  src/tests/credentials_tests.cpp ed50609cc5ed1937d2bcd782451245e0b5889ed5 
  src/tests/mesos.cpp adbb697c2bb908a584163dc9954b9e3138164433 
  src/tests/protobuf_io_tests.cpp 9263d8c2763cb30c93fd94be5344f83a6ce3e3b1 

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


Testing
-------


Thanks,

Michael Park

Re: Review Request 54603: Replaced `int` with `int_fd` in mesos.

Posted by Michael Park <mp...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54603/
-----------------------------------------------------------

(Updated Feb. 4, 2017, 5:39 p.m.)


Review request for mesos, Daniel Pravat and Joris Van Remoortere.


Changes
-------

Rebased.


Repository: mesos


Description
-------

Replaced `int` with `int_fd` in mesos.


Diffs (updated)
-----

  include/mesos/slave/container_logger.hpp 5781fecd08a2c92c3bc936598ca707a9a15f0e23 
  src/docker/docker.cpp b63b060ba1c1d39dc1702368cf667831edbd39bd 
  src/files/files.cpp 2b984ed73b8a5d416d873e8d7c10be09c9de5d8e 
  src/slave/containerizer/fetcher.cpp ba2e118bdefa932aa4109452548faf6cafb3451f 
  src/slave/containerizer/mesos/containerizer.hpp 9fcad933e1ec3b52d4dc366ba80d282deea04c0b 
  src/slave/containerizer/mesos/containerizer.cpp 4f0a773676da45fa40ad1ad9cdfab2a19249247d 
  src/slave/containerizer/mesos/io/switchboard.cpp 6de991781bd704a455f65fdac267bc16ea3f83c6 
  src/slave/containerizer/mesos/io/switchboard_main.cpp 87056c3f76525151834b4ff782e2009050941ccb 
  src/slave/containerizer/mesos/launch.hpp 5bba13998e879afb490c44c427dc6844c1c3c38d 
  src/slave/containerizer/mesos/launch.cpp 40116f989d99b87930aca323a656ce70bd241194 
  src/slave/state.cpp 6894875b37b7a34c8f3645087404584bbe982e6c 
  src/slave/status_update_manager.hpp 9fad0c51ec0fc00e134265850ad2236a585e46ec 
  src/slave/status_update_manager.cpp 056a684b52756d5c6309e7e2167a1532c4e60957 
  src/tests/containerizer/cgroups_tests.cpp 5c1ef70fb1feb76de602418286989b48563f26ff 
  src/tests/containerizer/io_switchboard_tests.cpp 738da5b6e1f4afdeea9ab081713dc88a5efe3953 
  src/tests/containerizer/memory_test_helper.cpp b94f0aac331da2e4f586245796176893195ad137 
  src/tests/containerizer/nested_mesos_containerizer_tests.cpp 9a6bcb54f77f8343786f42e3cb511f2e95aaff5e 
  src/tests/credentials_tests.cpp a6002528f7146e10a76fe49e3dcc5cba0d830605 
  src/tests/mesos.cpp adbb697c2bb908a584163dc9954b9e3138164433 
  src/tests/protobuf_io_tests.cpp 9263d8c2763cb30c93fd94be5344f83a6ce3e3b1 

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


Testing
-------


Thanks,

Michael Park