Review Request 40329: Removed stale comment.

2015-11-15 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40329/ --- Review request for mesos and Ben Mahler. Repository: mesos Description

Re: Review Request 38110: Quota: Checked sanity of quota set requests.

2015-11-15 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38110/#review106560 --- Ship it! src/master/master.hpp (line 879)

Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-15 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/ --- Review request for mesos, Benjamin Bannier and Joris Van Remoortere. Bugs:

Review Request 40327: mesos: Marked Boost as a "system" header.

2015-11-15 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40327/ --- Review request for mesos, Benjamin Bannier and Joris Van Remoortere. Bugs:

Re: Review Request 39230: Added prevention of SASL deprecation warnings all around its invocations on OS X.

2015-11-15 Thread Neil Conway
> On Nov. 2, 2015, 6:44 p.m., Alex Clemmer wrote: > > configure.ac, line 758 > > > > > > Sorry I'm super late to the party. Does it make sense to add these to > > the CMake build system as well? Using #pragma

Re: Review Request 40327: mesos: Marked Boost as a "system" header.

2015-11-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40327/#review106557 --- Patch looks great! Reviews applied: [40326, 40327] All tests

Re: Review Request 40211: Add mesos provisioner doc.

2015-11-15 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40211/ --- (Updated Nov. 15, 2015, 7:26 p.m.) Review request for mesos, Jie Yu and Jojy

Re: Review Request 39223: Added Quota Request Validation Tests.

2015-11-15 Thread Joerg Schad
> On Nov. 13, 2015, 2:23 a.m., Guangya Liu wrote: > > src/tests/master_quota_tests.cpp, lines 201-203 > > > > > > This format may be better? > >

Re: Review Request 40211: Add mesos provisioner doc.

2015-11-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40211/#review106569 --- Patch looks great! Reviews applied: [40211] All tests passed. -

Re: Review Request 40211: Add mesos provisioner doc.

2015-11-15 Thread Cong Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40211/#review106570 --- docs/mesos-provisioner.md (line 4)

Re: Review Request 40329: Removed stale comment.

2015-11-15 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40329/#review106594 --- Ship it! Ship It! - Guangya Liu On Nov. 15, 2015, 4:26 p.m.,

Review Request 40339: Added a flag to master to enable optimistic offers.

2015-11-15 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40339/ --- Review request for mesos, Artem Harutyunyan and Joris Van Remoortere. Bugs:

Re: Review Request 40326: libprocess: Marked Boost as a "system" header.

2015-11-15 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40326/#review106602 --- 3rdparty/libprocess/configure.ac (line 211)

Re: Review Request 40329: Removed stale comment.

2015-11-15 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40329/#review106562 --- Patch looks great! Reviews applied: [40329] All tests passed. -

Re: Review Request 39223: Added Quota Request Validation Tests.

2015-11-15 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39223/ --- (Updated Nov. 15, 2015, 6:05 p.m.) Review request for mesos, Alexander