Re: Review Request 50771: Added coverity build to build_docker.sh.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50771/#review144716 --- Patch looks great! Reviews applied: [50771] Passed command: expo

Re: Review Request 45960: Added interfaces to handle and track shareable resources.

2016-08-03 Thread Anindya Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45960/ --- (Updated Aug. 4, 2016, 5:05 a.m.) Review request for mesos and Jiang Yan Xu.

Re: Review Request 49862: Changed libmesos from static library to a shared library.

2016-08-03 Thread Srinivas Brahmaroutu
> On Aug. 2, 2016, 1:31 a.m., Alex Clemmer wrote: > > src/slave/cmake/SlaveConfigure.cmake, line 70 > > > > > > Oh. This is especially pertinent given that you seem to remove this > > line in #49874. I cannot recol

Re: Review Request 50763: Updated Linux 'MountInfoTable' entries to be sorted as expected.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50763/#review144713 --- Patch looks great! Reviews applied: [50762, 50763] Passed comman

Re: Review Request 50128: Added helper functions to 'Docker::Device'.

2016-08-03 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50128/#review144708 --- Can you please remove the review ID `50258, 50229` from the `Block

Re: Review Request 50128: Added helper functions to 'Docker::Device'.

2016-08-03 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50128/#review144706 --- src/docker/docker.hpp (lines 73 - 75)

Re: Review Request 50739: Updated '--fetcher_cache_dir' help string.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50739/#review144704 --- Bad patch! Reviews applied: [50739, 50737, 50736, 40512, 40268, 4

Re: Review Request 49851: Implemented `MemorySubsystem`.

2016-08-03 Thread Qian Zhang
> On July 27, 2016, 10:05 a.m., Qian Zhang wrote: > > src/slave/containerizer/mesos/isolators/cgroups/subsystem.cpp, line 494 > > > > > > How do we recover this field `updatedLimit`? I mean during agent > > recovery

Re: Review Request 49571: Added a benchmark test for allocations.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49571/#review144702 --- Bad patch! Reviews applied: [49571, 45967, 45966, 45964, 45963, 4

Re: Review Request 50775: Fixed sign comparisons in logrotate module.

2016-08-03 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50775/#review144699 --- Ship it! Ship It! - Jie Yu On Aug. 4, 2016, 12:05 a.m., Jose

Review Request 50775: Fixed sign comparisons in logrotate module.

2016-08-03 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50775/ --- Review request for mesos, Artem Harutyunyan and Joris Van Remoortere. Repositor

Re: Review Request 50010: Added example framework for handling inverse offers.

2016-08-03 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50010/ --- (Updated Aug. 3, 2016, 5:03 p.m.) Review request for mesos, Anand Mazumdar, Art

Review Request 50774: Added `-Wsign-compare` to global compiler flags.

2016-08-03 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50774/ --- Review request for mesos, Artem Harutyunyan, Joris Van Remoortere, and Michael P

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50723/ --- (Updated Aug. 3, 2016, 11:14 p.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Anand Mazumdar
> On Aug. 3, 2016, 7:58 p.m., Adam B wrote: > > src/master/master.cpp, lines 5164-5166 > > > > > > Maybe increment `metrics->invalid_status_updates` in the else case here? > > And log here, as before? > > Anand

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50723/#review144677 --- Fix it, then Ship it! src/master/master.cpp (lines 5201 - 5203

Re: Review Request 50767: Updated logrotation module to use `os::pagesize()`.

2016-08-03 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50767/ --- (Updated Aug. 3, 2016, 10:19 p.m.) Review request for mesos, Benjamin Mahler an

Re: Review Request 50713: Documented behavior of content-type header in presence of errors.

2016-08-03 Thread Abhishek Dasgupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50713/ --- (Updated Aug. 3, 2016, 10:12 p.m.) Review request for mesos and Anand Mazumdar.

Re: Review Request 50640: Improved HTTP API docs.

2016-08-03 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50640/#review144683 --- Ship it! LGTM - Anand Mazumdar On July 31, 2016, 2:53 p.m.,

Re: Review Request 50641: Fixed typo in comment.

2016-08-03 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50641/#review144682 --- Ship it! Ship It! - Anand Mazumdar On July 31, 2016, 2:53 p.

Re: Review Request 50762: Updated 'NvidiaVolume' to not error out when binary is missing.

2016-08-03 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50762/#review144680 --- Ship it! Ship It! - Benjamin Mahler On Aug. 3, 2016, 7:10 p.

Re: Review Request 50766: Added flag in logrotate module to control number of libprocess threads.

2016-08-03 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50766/#review144679 --- Ship it! Ship It! - Joseph Wu On Aug. 3, 2016, 2:16 p.m., Jo

Re: Review Request 50767: Updated logrotation module to use `os::pagesize()`.

2016-08-03 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50767/#review144672 --- Fix it, then Ship it! It doesn't compile on Windows just yet, b

Review Request 50769: Fixed a broken link to CNI repo.

2016-08-03 Thread Avinash sridharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50769/ --- Review request for mesos and Jie Yu. Bugs: MESOS-5985 https://issues.apache

Re: Review Request 50766: Added flag in logrotate module to control number of libprocess threads.

2016-08-03 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50766/ --- (Updated Aug. 3, 2016, 9:16 p.m.) Review request for mesos, Benjamin Mahler and

Review Request 50771: Added coverity build to build_docker.sh.

2016-08-03 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50771/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-5984 https://issues.ap

Re: Review Request 50733: Removed CgroupsCpushareIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50733/ --- (Updated Aug. 3, 2016, 2:18 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50766: Added flag in logrotate module to control number of libprocess threads.

2016-08-03 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50766/#review144668 --- src/slave/container_loggers/lib_logrotate.hpp (lines 118 - 119) <

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Adam B
> On Aug. 3, 2016, 12:58 p.m., Adam B wrote: > > src/master/master.cpp, lines 5164-5166 > > > > > > Maybe increment `metrics->invalid_status_updates` in the else case here? > > And log here, as before? > > Anan

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Anand Mazumdar
> On Aug. 3, 2016, 7:58 p.m., Adam B wrote: > > src/master/master.cpp, lines 5164-5166 > > > > > > Maybe increment `metrics->invalid_status_updates` in the else case here? > > And log here, as before? This was

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50723/ --- (Updated Aug. 3, 2016, 8:44 p.m.) Review request for mesos and Vinod Kone. Ch

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Anand Mazumdar
> On Aug. 3, 2016, 6:12 p.m., Neil Conway wrote: > > src/master/master.cpp, line 5142 > > > > > > Seems like we should still be logging when we get a status update from > > an unknown framework. +1, my bad. > On

Review Request 50767: Updated logrotation module to use `os::pagesize()`.

2016-08-03 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50767/ --- Review request for mesos, Benjamin Mahler and Joseph Wu. Repository: mesos De

Review Request 50766: Added flag in logrotate module to control number of libprocess threads.

2016-08-03 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50766/ --- Review request for mesos, Benjamin Mahler and Joseph Wu. Bugs: MESOS-5983 h

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50723/#review144662 --- src/master/master.cpp (lines 5154 - 5156)

Re: Review Request 49736: Added PATCH_CMD in 3rdparty/CMakeLists.txt for ELFIO.

2016-08-03 Thread Kevin Klues
> On July 12, 2016, 4:40 p.m., Alex Clemmer wrote: > > 3rdparty/CMakeLists.txt, line 141 > > > > > > This probably won't work on Windows. You will notice that for (_e.g._) > > `HTTP_PARSER_PATCH_CMD`, we have someth

Review Request 50762: Updated 'NvidiaVolume' to not error out when binary is missing.

2016-08-03 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50762/ --- Review request for mesos, Benjamin Mahler and Jie Yu. Bugs: MESOS-5982 http

Review Request 50763: Updated Linux 'MountInfoTable' entries to be sorted as expected.

2016-08-03 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50763/ --- Review request for mesos, Benjamin Mahler and Jie Yu. Bugs: MESOS-5969 http

Re: Review Request 45964: Add unit tests for sharing of resources.

2016-08-03 Thread Anindya Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45964/ --- (Updated Aug. 3, 2016, 7 p.m.) Review request for mesos and Jiang Yan Xu. Cha

Review Request 50739: Updated '--fetcher_cache_dir' help string.

2016-08-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50739/ --- Review request for mesos and Jie Yu. Repository: mesos Description --- N

Re: Review Request 49571: Added a benchmark test for allocations.

2016-08-03 Thread Anindya Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49571/#review144158 --- src/tests/hierarchical_allocator_tests.cpp (line 252)

Re: Review Request 50738: Improved performance in addition and subtraction of resources.

2016-08-03 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50738/#review144639 --- Ship it! Ship It! - Jiang Yan Xu On Aug. 3, 2016, 12:36 a.m.

Re: Review Request 49855: Enabled cgroups unified isolator in isolation.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49855/#review144638 --- Patch looks great! Reviews applied: [49819, 49820, 49821, 49823,

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50723/#review144633 --- src/master/master.cpp

Re: Review Request 49854: Implemented `DevicesSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49854/ --- (Updated Aug. 3, 2016, 5:59 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49853: Implemented `PerfEventSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49853/ --- (Updated Aug. 3, 2016, 5:59 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49850: Implemented `CpuacctSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49850/ --- (Updated Aug. 3, 2016, 5:57 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49849: Implemented `CpuSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49849/ --- (Updated Aug. 3, 2016, 5:57 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49852: Implemented `NetClsSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49852/ --- (Updated Aug. 3, 2016, 5:58 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49851: Implemented `MemorySubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49851/ --- (Updated Aug. 3, 2016, 5:58 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50737: Parametrized libprocess HTTPTests by SSL configuration.

2016-08-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50737/ --- (Updated Aug. 3, 2016, 5:37 p.m.) Review request for mesos, Anand Mazumdar, Jos

Review Request 50737: Parametrized libprocess HTTPTests by SSL configuration.

2016-08-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50737/ --- Review request for mesos, Anand Mazumdar, Joseph Wu, and Vinod Kone. Bugs: MESO

Re: Review Request 49852: Implemented `NetClsSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49852/ --- (Updated Aug. 3, 2016, 5:30 p.m.) Review request for mesos, Gilbert Song, Jie Y

Review Request 50736: Added SSL support to libprocess HTTP request helpers.

2016-08-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50736/ --- Review request for mesos, Joseph Wu and Vinod Kone. Bugs: MESOS-5966 https:

Re: Review Request 50750: Removed CgroupsNetClsIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50750/ --- (Updated Aug. 3, 2016, 5:32 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50751: Removed CgroupsPerfEventIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50751/ --- (Updated Aug. 3, 2016, 5:32 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50749: Removed CgroupsDevicesIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50749/ --- (Updated Aug. 3, 2016, 5:31 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49853: Implemented `PerfEventSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49853/ --- (Updated Aug. 3, 2016, 5:31 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49854: Implemented `DevicesSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49854/ --- (Updated Aug. 3, 2016, 5:31 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50733: Removed CgroupsCpushareIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50733/ --- (Updated Aug. 3, 2016, 5:31 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50748: Removed CgroupsMemIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50748/ --- (Updated Aug. 3, 2016, 5:31 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49855: Enabled cgroups unified isolator in isolation.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49855/ --- (Updated Aug. 3, 2016, 5:31 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 45573: Added `PerfEventHandleManager`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45573/ --- (Updated Aug. 3, 2016, 5:31 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49852: Implemented `NetClsSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49852/ --- (Updated Aug. 3, 2016, 5:17 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49851: Implemented `MemorySubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49851/ --- (Updated Aug. 3, 2016, 5:16 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49850: Implemented `CpuacctSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49850/ --- (Updated Aug. 3, 2016, 5:15 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49849: Implemented `CpuSubsystem`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49849/ --- (Updated Aug. 3, 2016, 5:14 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49828: Added default methods implementations for `Subsystem` base class.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49828/ --- (Updated Aug. 3, 2016, 5:13 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49817: Implemented `CgroupsIsolatorProcess::recover`.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49817/ --- (Updated Aug. 3, 2016, 5:12 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 49862: Changed libmesos from static library to a shared library.

2016-08-03 Thread Srinivas Brahmaroutu
> On Aug. 2, 2016, 1:29 a.m., Alex Clemmer wrote: > > 3rdparty/cmake/Mesos3rdpartyConfigure.cmake, line 51 > > > > > > You're right to split this out, but just for my own education, was > > there a specific reason y

Re: Review Request 49862: Changed libmesos from static library to a shared library.

2016-08-03 Thread Srinivas Brahmaroutu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49862/ --- (Updated Aug. 3, 2016, 4:40 p.m.) Review request for mesos, Alex Clemmer and Jo

Re: Review Request 49736: Added PATCH_CMD in 3rdparty/CMakeLists.txt for ELFIO.

2016-08-03 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49736/ --- (Updated Aug. 3, 2016, 9:21 a.m.) Review request for mesos and Joseph Wu. Cha

Re: Review Request 50751: Removed CgroupsPerfEventIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50751/ --- (Updated Aug. 3, 2016, 2:36 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50749: Removed CgroupsDevicesIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50749/ --- (Updated Aug. 3, 2016, 2:35 p.m.) Review request for mesos, Gilbert Song, Jie Y

Re: Review Request 50751: Removed CgroupsPerfEventIsolatorProcess.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50751/#review144614 --- Bad patch! Reviews applied: [50751, 50750, 50749, 50748, 50733, 4

Review Request 50751: Removed CgroupsPerfEventIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50751/ --- Review request for mesos, Gilbert Song, Jie Yu, Qian Zhang, and Zhengju Sha. Bu

Review Request 50750: Removed CgroupsNetClsIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50750/ --- Review request for mesos, Gilbert Song, Jie Yu, Qian Zhang, and Zhengju Sha. Bu

Review Request 50749: Removed CgroupsDevicesIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50749/ --- Review request for mesos, Gilbert Song, Jie Yu, Qian Zhang, and Zhengju Sha. Bu

Review Request 50748: Removed CgroupsMemIsolatorProcess.

2016-08-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50748/ --- Review request for mesos, Gilbert Song, Jie Yu, Qian Zhang, and Zhengju Sha. Bu

Re: Review Request 49360: Supported TCP check in health check.

2016-08-03 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49360/#review144613 --- src/health-check/health_checker.hpp (line 316)

Re: Review Request 36816: Supported HTTP/HTTPS in health check.

2016-08-03 Thread Gastón Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36816/#review144598 --- include/mesos/v1/mesos.proto (line 339)

Re: Review Request 50738: Improved performance in addition and subtraction of resources.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50738/#review144612 --- Patch looks great! Reviews applied: [50738] Passed command: expo

Re: Review Request 50563: Made HierarchicalAllocatorProcess specify a process ID.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50563/#review144611 --- Ship it! Ship It! - Alexander Rukletsov On Aug. 2, 2016, 1:5

Re: Review Request 50521: Added "zookeeper" prefix to ZK process IDs.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50521/#review144610 --- Ship it! Ship It! - Alexander Rukletsov On Aug. 2, 2016, 1:5

Re: Review Request 50639: Made all Actor IDs outside of libprocess follow the same schema.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50639/#review144609 --- Ship it! Ship It! - Alexander Rukletsov On Aug. 2, 2016, 1:5

Re: Review Request 50519: Made libprocess Actor IDs consistent.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50519/#review144608 --- Ship it! Ship It! - Alexander Rukletsov On Aug. 2, 2016, 1:5

Re: Review Request 50515: Added missing process IDs.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50515/#review144603 --- Fix it, then Ship it! I'll fix the outstanding issues and will

Re: Review Request 50637: Added id to the libprocess Sequence constructor.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50637/#review144606 --- Ship it! Ship It! - Alexander Rukletsov On Aug. 2, 2016, 1:4

Re: Review Request 50513: Added missing process IDs in libprocess.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50513/#review144607 --- Ship it! Ship It! - Alexander Rukletsov On Aug. 2, 2016, 1:4

Re: Review Request 50513: Added missing process IDs in libprocess.

2016-08-03 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50513/#review144599 --- 3rdparty/libprocess/include/process/collect.hpp (line 136)

Re: Review Request 50733: Removed CgroupsCpushareIsolatorProcess.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50733/#review144604 --- Patch looks great! Reviews applied: [49819, 49820, 49821, 49823,

Re: Review Request 50609: Moved check for '--version' in master and agent 'main.cpp'.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50609/#review144596 --- Patch looks great! Reviews applied: [50570, 50609] Passed comman

Re: Review Request 50723: Fixed the master to recover resources/update state for orphan tasks.

2016-08-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50723/#review144591 --- Patch looks great! Reviews applied: [50723] Passed command: expo

Review Request 50738: Improved performance in addition and subtraction of resources.

2016-08-03 Thread Anindya Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50738/ --- Review request for mesos and Jiang Yan Xu. Bugs: MESOS-5978 https://issues.