Review Request 41705: WIP: Added support for enforcing quota on (persistent) volumes (MESOS-4198).

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41705/ --- Review request for mesos and Jie Yu. Bugs: MESOS-4198 https://issues.apache

Review Request 41704: WIP: Added support for checking whether a given path is absolute.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41704/ --- Review request for mesos and Jie Yu. Bugs: MESOS-4198 https://issues.apache

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/#review111824 --- Ship it! Ship It! - Qian Zhang On Dec. 24, 2015, 10:46 a.m., Yo

Re: Review Request 41672: Test case(s) for weights + allocation behavior

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41672/#review111819 --- Patch looks great! Reviews applied: [41597, 41672] Passed command

Re: Review Request 41672: Test case(s) for weights + allocation behavior

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41672/ --- (Updated Dec. 24, 2015, 2:49 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/ --- (Updated Dec. 24, 2015, 2:46 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/ --- (Updated Dec. 24, 2015, 2:41 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/#review111818 --- src/master/allocator/mesos/hierarchical.cpp (line 1055)

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/ --- (Updated Dec. 24, 2015, 1:42 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

> On Dec. 10, 2015, 4:12 a.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/rmdir.hpp, line > > 109 > > > > > > Can you explain / comment why here (as well as on L139)

Re: Review Request 41680: Reduced LogLevel in order to avoid overflowing logs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41680/#review111815 --- Ship it! Ship It! - Cody Maloney On Dec. 23, 2015, 9:06 a.m., J

Re: Review Request 41697: Fixed a documentation bug in containerizer internals doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41697/#review111813 --- Patch looks great! Reviews applied: [41697] Passed command: expor

Review Request 41697: Fixed a documentation bug in containerizer internals doc.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41697/ --- Review request for mesos, Jie Yu and Jojy Varghese. Repository: mesos Descrip

Re: Review Request 41425: Fix the Mesos pkg-config.

> On Dec. 23, 2015, 9:15 p.m., Alex Clemmer wrote: > > Just as a short note here, `pkg-config` doesn't work well with the rest of > > the Windows tool chain, so it's worth wondering what our distribution story > > should be there. > > > > If the consumer is using CMake, then this information i

Re: Review Request 41424: Install picojson.h if we are using the bundled version.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41424/#review111807 --- Ship it! Much in the spirit of my review of #41425, it's interesti

Re: Review Request 41425: Fix the Mesos pkg-config.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41425/#review111806 --- Ship it! Just as a short note here, `pkg-config` doesn't work well

Re: Review Request 41593: stout: Added `jsonify` function.

> On Dec. 22, 2015, 4:52 a.m., Benjamin Hindman wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/jsonify.hpp, line 536 > > > > > > Do you think the comment over `WriterProxy` will be sufficient to > > cap

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

> On Dec. 15, 2015, 3:54 a.m., Alex Naparu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/rmdir.hpp, line > > 51 > > > > > > Calling FindClose on an invalid handle is not the best idea. At bes

Re: Review Request 41593: stout: Added `jsonify` function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41593/ --- (Updated Dec. 23, 2015, 5:57 p.m.) Review request for mesos and Benjamin Hindma

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39584/#review111775 --- Bad patch! Reviews applied: [39621, 39620, 39583, 39584] Failed c

Re: Review Request 41681: Introduce HTTP endpoint /weights for updating weight.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41681/#review111770 --- Bad patch! Reviews applied: [41597, 41681] Failed command: ./supp

Re: Review Request 41680: Reduced LogLevel in order to avoid overflowing logs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41680/#review111767 --- Patch looks great! Reviews applied: [41680] Passed command: expor

Re: Review Request 39584: Windows: Implemented `os::rmdir.hpp`.

> On Dec. 15, 2015, 3:54 a.m., Alex Naparu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/rmdir.hpp, line > > 39 > > > > > > What if the path ends with multiple '\' chars to begin with? Good

Re: Review Request 41672: Test case(s) for weights + allocation behavior

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41672/ --- (Updated Dec. 23, 2015, 9:44 a.m.) Review request for mesos, Adam B and Neil Co

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/ --- (Updated Dec. 23, 2015, 9:44 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 41682: Reflected deprecation of external containerizer in documentation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41682/#review111761 --- Ship it! Ship It! - Bernd Mathiske On Dec. 23, 2015, 1:30 a.m.,

Re: Review Request 41683: Added links to individual containerizers in containerizer-internal.md.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41683/#review111760 --- Ship it! Ship It! - Bernd Mathiske On Dec. 23, 2015, 1:32 a.m.,

Re: Review Request 41680: Reduced LogLevel in order to avoid overflowing logs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41680/#review111757 --- Ship it! Ship It! - Bernd Mathiske On Dec. 23, 2015, 1:06 a.m.,

Review Request 41683: Added links to individual containerizers in containerizer-internal.md.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41683/ --- Review request for mesos, Bernd Mathiske and Jojy Varghese. Bugs: MESOS-3370

Review Request 41682: Reflected deprecation of external containerizer in documentation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41682/ --- Review request for mesos and Bernd Mathiske. Bugs: MESOS-3370 https://issue

Review Request 41681: Introduce HTTP endpoint /weights for updating weight.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41681/ --- Review request for mesos, Adam B, Neil Conway, and Qian Zhang. Bugs: MESOS-4214

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/ --- (Updated Dec. 23, 2015, 9:18 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 41597: Extending allocator interface to support dynamic weights

> On Dec. 23, 2015, 8:22 a.m., Qian Zhang wrote: > > src/master/allocator/mesos/hierarchical.cpp, line 1057 > > > > > > Do we support updating weight for an inactive role? If yes, I think > > this `CHECK` will fail

Review Request 41680: Reduced LogLevel in order to avoid overflowing logs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41680/ --- Review request for mesos and Bernd Mathiske. Bugs: MESOS-4181 https://issue

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/ --- (Updated Dec. 23, 2015, 9:05 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 41597: Extending allocator interface to support dynamic weights

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41597/#review111749 --- src/master/allocator/mesos/hierarchical.cpp (line 1051)