Re: Review Request 35433: CHECK that checkpointed resources exist on the slave.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 12:39 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35433

Re: Review Request 35438: Fixed fetcher cache test race for resource offers when starting tasks and changed corresponding CHECK to EXPECT.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 2:28 p.m., Bernd Mathiske wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35438

Re: Review Request 35455: Update 'Getting Started' documentation.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 12:58 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35455

Re: Review Request 35494: Added format for template declaration in the styleguide.

2015-06-16 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 16, 2015, 11:27 a.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35494

Re: Review Request 34976: Added installation instructions for Ubuntu 14.04 and OSX

2015-06-12 Thread Mesos ReviewBot
can add `-j=number of cores V=0` Failed to commit patch - Mesos ReviewBot On June 12, 2015, 6:23 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34976

Re: Review Request 35433: CHECK that checkpointed resources exist on the slave.

2015-06-13 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 13, 2015, 9:43 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35433/ --- (Updated

Re: Review Request 34375: Removed use of namespace aliases.

2015-06-13 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 13, 2015, 9:51 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34375/ --- (Updated

Re: Review Request 35426: mesos: Use CHECK_SOME, CHECK_NONE, CHECK_ERROR.

2015-06-13 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On June 13, 2015, 2:07 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35426

Re: Review Request 35423: libprocess: Rebased for _CheckFatal changes in stout.

2015-06-13 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 13, 2015, 12:24 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35423

Re: Review Request 35129: Refactor Future::Data to use ResultT. Remove dynamic allocation.

2015-06-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 14, 2015, 4:17 p.m., Joris Van Remoortere wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35129

Re: Review Request 35438: Fixed fetcher cache test race for resource offers when starting tasks and changed corresponding CHECK to EXPECT.

2015-06-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 14, 2015, 1:54 p.m., Bernd Mathiske wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35438

Re: Review Request 35441: Removed unused macros.

2015-06-14 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 14, 2015, 5:26 p.m., Benjamin Hindman wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35441

Re: Review Request 35571: Adding ability to decode JSON from ZK

2015-06-17 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 17, 2015, 6:18 p.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35571

Re: Review Request 35586: Updated LinuxLauncher to receive list of namespaces.

2015-06-17 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 18, 2015, 1:23 a.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35586

Re: Review Request 35571: Adding ability to decode JSON from ZK

2015-06-17 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 18, 2015, 2:27 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35571

Re: Review Request 35510: Introduced General and Markdown Documentation Style Guides.

2015-06-17 Thread Mesos ReviewBot
: 3rdparty/libprocess/README.md stout: 3rdparty/libprocess/3rdparty/stout/README.md Failed to commit patch - Mesos ReviewBot On June 17, 2015, 6:52 a.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 35510: Introduced General and Markdown Documentation Style Guides.

2015-06-17 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 17, 2015, 8:22 a.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35510

Re: Review Request 35553: Rename libprocess User Guide to Developer Guide.

2015-06-17 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 17, 2015, 9:22 a.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35553/ --- (Updated

Re: Review Request 35544: containerizer: statically initialize isolator factories

2015-06-16 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 17, 2015, 12:13 a.m., Jojy Varghese wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35544

Re: Review Request 35510: Introduced General and Markdown Documentation Style Guides.

2015-06-16 Thread Mesos ReviewBot
Style Guides. Introduced General and Markdown Documentation Style Guides. Review: https://reviews.apache.org/r/35510 docs/mesos-markdown-style-guide.md:123: new blank line at EOF. Failed to commit patch - Mesos ReviewBot On June 16, 2015, 6:10 p.m., Joerg Schad wrote

Re: Review Request 30032: Added support for cache control in libprocess when dealing with static files.

2015-06-17 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 17, 2015, 3:42 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30032

Re: Review Request 35568: Remove html from libprocess Developer Guide.

2015-06-17 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 17, 2015, 3:24 p.m., Joerg Schad wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35568

Re: Review Request 34721: Added QoS kill executor correction test.

2015-06-12 Thread Mesos ReviewBot
, 34719, 34720, 34721] All tests passed. - Mesos ReviewBot On June 12, 2015, 10:37 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34721

Re: Review Request 14460: Add debian directory to create mesos packages

2015-06-12 Thread Mesos ReviewBot
/control:83: trailing whitespace. + ${misc:Depends}, debian/control:104: new blank line at EOF. debian/mesos-master.init:75: new blank line at EOF. debian/mesos-slave.default:13: new blank line at EOF. debian/mesos-slave.init:73: new blank line at EOF. Failed to commit patch - Mesos ReviewBot

Re: Review Request 14309: Properly get gethostbyname2_r error on OS X.

2015-06-12 Thread Mesos ReviewBot
Failed to apply patch - Mesos ReviewBot On Sept. 24, 2013, 6:03 a.m., Benjamin Hindman wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14309

Re: Review Request 35404: Fixed http::Request::client to be set correctly.

2015-06-12 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 12, 2015, 10:40 p.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35404

Re: Review Request 14960: implementation of CLI mesos-status

2015-06-12 Thread Mesos ReviewBot
. - Mesos ReviewBot On Nov. 19, 2013, 11:47 p.m., Du Li wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14960/ --- (Updated Nov

Re: Review Request 35253: Enable configure to detect libevent and openssl on Mac OS X.

2015-06-13 Thread Mesos ReviewBot
, 29530, 29531, 29532, 29533, 29534, 29535, 29536, 29538, 29539, 29540, 29569, 28763, 35253] All tests passed. - Mesos ReviewBot On June 13, 2015, 2:32 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-10 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 11, 2015, 12:04 a.m., Lily Chen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35330/ --- (Updated

Re: Review Request 34911: Added a test for launching an executor on revocable resources.

2015-06-10 Thread Mesos ReviewBot
: ./support/apply-review.sh -n -r 34910 Error: 2015-06-10 23:34:43 URL:https://reviews.apache.org/r/34910/diff/raw/ [8902/8902] - 34910.patch [1] error: patch failed: src/slave/slave.cpp:3098 error: src/slave/slave.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On June 10, 2015, 11

Re: Review Request 35431: Enable Option to hold a noncopyable type.

2015-06-13 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 13, 2015, 8:57 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35431

Re: Review Request 30032: Added support for cache control in libprocess when dealing with static files.

2015-06-15 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 15, 2015, 3:27 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30032

Re: Review Request 35467: Fix a comment in Slave.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 5:37 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35467/ --- (Updated

Re: Review Request 34835: Add constexpr to C++ whitelist

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 5:59 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34835/ --- (Updated

Re: Review Request 35179: MESOS-1733 Variadic Path Join

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 3:25 p.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35179

Re: Review Request 35470: Fixed the flaky oversbuscription tests.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 7:02 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35470/ --- (Updated June

Re: Review Request 35341: Added Uber to the Powered-by-Mesos page

2015-06-10 Thread Mesos ReviewBot
to the Powered-by-Mesos page Review: https://reviews.apache.org/r/35341 docs/powered-by-mesos.md:91: new blank line at EOF. Failed to commit patch - Mesos ReviewBot On June 11, 2015, 2:41 a.m., Marco Massenzio wrote: --- This is an automatically

Re: Review Request 35333: Small fix in updateSlave() to make resource math clearer.

2015-06-10 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 11, 2015, 12:07 a.m., Jiang Yan Xu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35333

Re: Review Request 35332: Add a test case for egress flow ID recovery

2015-06-10 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 11, 2015, 1:30 a.m., Cong Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35332

Re: Review Request 31915: MemIsolator: Improved some statistics naming. (MESOS-2104)

2015-06-11 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 11, 2015, 10:51 p.m., Chi Zhang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31915

Re: Review Request 34721: Added QoS kill executor correction test.

2015-06-11 Thread Mesos ReviewBot
, 34719, 34720, 34721] All tests passed. - Mesos ReviewBot On June 11, 2015, 10:58 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34721

Re: Review Request 13620: Fix the Allocator to recover resources when a slave/framework is removed

2015-06-11 Thread Mesos ReviewBot
not apply Failed to apply patch - Mesos ReviewBot On Aug. 19, 2013, 8:39 p.m., Thomas Marshall wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13620

Re: Review Request 35473: Removed a few incorrect CHECKs in DRF sorter.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 10:03 p.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35473/ --- (Updated June

Re: Review Request 35179: MESOS-1733 Variadic Path Join

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 11:23 p.m., Anand Mazumdar wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35179

Re: Review Request 35432: Enable move semantics for Future and Promise.

2015-06-13 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 13, 2015, 9:26 p.m., Michael Park wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35432/ --- (Updated

Re: Review Request 30339: Call hookmanager only if some hooks are installed.

2015-06-15 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 15, 2015, 10:26 p.m., Kapil Arya wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30339/ --- (Updated

Re: Review Request 35482: Replace adhoc JSON conversion functions for ResourceStatistics with a protocol buffer to JSON converter.

2015-06-15 Thread Mesos ReviewBot
/apply-review.sh -n -r 35482 Error: 2015-06-16 00:15:18 URL:https://reviews.apache.org/r/35482/diff/raw/ [44012/44012] - 35482.patch [1] error: patch failed: src/Makefile.am:353 error: src/Makefile.am: patch does not apply Failed to apply patch - Mesos ReviewBot On June 16, 2015, 12:14 a.m., Paul

Re: Review Request 34970: Added an example framework for oversubscribing resources.

2015-06-02 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 3, 2015, 1:21 a.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34970

Re: Review Request 34662: Modularized ResourceEstimator and added test for RE module

2015-05-27 Thread Mesos ReviewBot
distcheck Error: make dist-gzip am__post_remove_distdir='@:' make[1]: Entering directory `/home/jenkins/jenkins-slave/workspace/mesos-reviewbot' if test -d mesos-0.23.0; then find mesos-0.23.0 -type d ! -perm -200 -exec chmod u+w {} ';' rm -rf mesos-0.23.0 || { sleep 5 rm -rf mesos-0.23.0

Re: Review Request 34676: Add printing of extended attributes to Resource objects.

2015-05-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 26, 2015, 11:19 p.m., Brian Wickman wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34676

Re: Review Request 34529: Add non-const reference version of OptionT::get.

2015-05-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 27, 2015, 6:40 a.m., Mark Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34529/ --- (Updated May

Re: Review Request 34687: (De)Serializing MasterInfo PB to JSON

2015-05-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 26, 2015, 11:54 p.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34687

Re: Review Request 34782: Convert Non-POD static variables in fq_codel and ingress to constexpr

2015-05-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 28, 2015, 8:28 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34782/ --- (Updated

Re: Review Request 34748: Defined protobuf for usage returned by Resource Monitor. Reused ResourceUsage. Pass ResourceUsage to ResourceEstimator

2015-05-28 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 28, 2015, 6:41 p.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34748

Re: Review Request 34736: Implemented 'updateSlave()' call in the master.

2015-05-28 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 29, 2015, 12:31 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34736

Re: Review Request 34195: Refactoring to use FlagsBase common functionality

2015-05-28 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 29, 2015, 1:10 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34195

Re: Review Request 29507: Added Configurable Slave Ping Timeouts

2015-05-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 28, 2015, 11:13 p.m., Adam B wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29507/ --- (Updated May

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 26, 2015, 10:37 a.m., Alexander Rukletsov wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545

Re: Review Request 34545: Updated the allocator related docs.

2015-05-29 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 29, 2015, 12:48 p.m., Alexander Rukletsov wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34545

Re: Review Request 30612: Added /master/frameworks/{framework}/tasks/{task} endpoint.

2015-05-29 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 29, 2015, 2:47 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30612

Re: Review Request 33793: HTTP headers should be considered case-insensitive.

2015-05-31 Thread Mesos ReviewBot
not apply Failed to apply patch - Mesos ReviewBot On May 31, 2015, 6:14 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33793

Re: Review Request 34863: Add tests for new qdisc statistics functions.

2015-05-31 Thread Mesos ReviewBot
] All tests passed. - Mesos ReviewBot On May 31, 2015, 8:02 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34863

Re: Review Request 34872: let libprocess to compile on arm64 servers

2015-05-31 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 1, 2015, 5:06 a.m., Dong Robin wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34872/ --- (Updated

Re: Review Request 33792: Extend hashmap to support custom equality and hash

2015-06-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 1, 2015, 4:17 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33792

Re: Review Request 34887: MESOS-2788: Fixed the statistics got by cmd *mesos ps* and webUI is mis-matching

2015-06-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 1, 2015, 12:15 p.m., weitao zhou wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34887/ --- (Updated

Re: Review Request 34882: let mesos to compile and run on arm64 servers

2015-06-01 Thread Mesos ReviewBot
. +- lock xadd dword ptr [eax], ebx; 3rdparty/zookeeper-3.4.5.patch:131: trailing whitespace. +-mov result, ecx; // result = ebx; 3rdparty/zookeeper-3.4.5.patch:133: trailing whitespace. +- return result; Failed to commit patch - Mesos ReviewBot On June 1, 2015, 7:11 a.m., Dong

Re: Review Request 34646: Redirect to the leader master when current master is not a leader.

2015-06-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 1, 2015, 6:35 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34646

Re: Review Request 34894: Add new message for Traffic Control statistics

2015-06-01 Thread Mesos ReviewBot
, 34894] All tests passed. - Mesos ReviewBot On June 1, 2015, 4:40 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34894

Re: Review Request 34362: Include ExecutorInfos in master/state.json

2015-06-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 1, 2015, 5:13 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34362

Re: Review Request 34645: Update existing lambdas to meet style guide

2015-06-01 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 1, 2015, 5:26 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34645

Re: Review Request 34908: Rename --docker_sandbox_directory flag for general use.

2015-06-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 1, 2015, 9:48 p.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34908/ --- (Updated

Re: Review Request 31444: Support chrooting in MesosContainerizer launch helper.

2015-06-01 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 1, 2015, 9:50 p.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31444

Re: Review Request 34431: Add htb queueing discipline

2015-05-27 Thread Mesos ReviewBot
/tests/routing_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot On May 22, 2015, 5:10 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34431

Re: Review Request 34662: Modularized ResourceEstimator and added test for RE module

2015-05-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 27, 2015, 4:34 p.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34662/ --- (Updated

Re: Review Request 34703: Added stream manipulators for the Time object.

2015-05-27 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 27, 2015, 1:33 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34703

Re: Review Request 30032: Added support for cache control in libprocess when dealing with static files.

2015-05-27 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 27, 2015, 2:50 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30032

Re: Review Request 30032: Added support for cache control in libprocess when dealing with static files.

2015-05-26 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 26, 2015, 2:41 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30032

Re: Review Request 34669: Add a virtualenv bootstrapper for post-reviews.

2015-05-26 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 26, 2015, 5:30 p.m., Brian Wickman wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34669/ --- (Updated

Re: Review Request 34835: Add constexpr to C++ whitelist

2015-05-29 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 29, 2015, 11:23 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34835/ --- (Updated

Re: Review Request 34832: Add new qdisc tests

2015-05-29 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On May 29, 2015, 10:09 p.m., Paul Brett wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34832

Re: Review Request 34529: Add non-const reference version of OptionT::get.

2015-05-30 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 31, 2015, 1:13 a.m., Mark Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34529/ --- (Updated May

Re: Review Request 34687: (De)Serializing MasterInfo PB to JSON

2015-05-30 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 31, 2015, 2:58 a.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34687

Re: Review Request 34529: Add non-const reference version of OptionT::get.

2015-05-30 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 31, 2015, 4:39 a.m., Mark Wang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34529/ --- (Updated May

Re: Review Request 34737: Added test to verify fix for MESOS-2771

2015-05-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 27, 2015, 11:14 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34737

Re: Review Request 34724: Doxygen enhancements

2015-05-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 27, 2015, 6:38 p.m., Niklas Nielsen wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34724

Re: Review Request 34687: (De)Serializing MasterInfo PB to JSON

2015-05-27 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 27, 2015, 6:07 p.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34687

Re: Review Request 32664: Add port mapping isolator statistics tests

2015-05-27 Thread Mesos ReviewBot
/apply-review.sh -n -r 32664 Error: 2015-05-28 05:19:18 URL:https://reviews.apache.org/r/32664/diff/raw/ [42205/42205] - 32664.patch [1] error: patch failed: src/tests/port_mapping_tests.cpp:2015 error: src/tests/port_mapping_tests.cpp: patch does not apply Failed to apply patch - Mesos ReviewBot

Re: Review Request 34662: Modularized ResourceEstimator and added test for RE module

2015-05-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 28, 2015, 1:20 a.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34662/ --- (Updated

Re: Review Request 34655: Use relative url in /help generated links point

2015-05-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 28, 2015, 7:54 a.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34655

Re: Review Request 34748: Defined protobuf for usage returned by Resource Monitor. Reused ResourceUsage.

2015-05-28 Thread Mesos ReviewBot
. - Mesos ReviewBot On May 28, 2015, 2:37 a.m., Bartek Plotka wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34748/ --- (Updated

Re: Review Request 34886: MESOS-2787: Fixed the exception KeyError: 'mem_rss_bytes' when run command mesos-ps

2015-06-02 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 2, 2015, 2:29 a.m., weitao zhou wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34886/ --- (Updated

Re: Review Request 33358: Moved implementation of StatusUpdateStream to a compilation unit.

2015-06-02 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 2, 2015, 10:24 a.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33358

Re: Review Request 33825: Added executor implementation for the new HTTP API

2015-06-02 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On June 2, 2015, 2:06 p.m., Alexander Rojas wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33825

Re: Review Request 34944: Used flags validation to handle --help.

2015-06-02 Thread Mesos ReviewBot
passed. - Mesos ReviewBot On June 2, 2015, 2:46 p.m., Benjamin Hindman wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34944

Re: Review Request 34310: Use IDLE scheduling for revocable CPU in cgroups isolator.

2015-05-29 Thread Mesos ReviewBot
tests passed. - Mesos ReviewBot On May 29, 2015, 7:57 p.m., Ian Downes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34310

Re: Review Request 36079: Added Subscribe call support to the master and the C++ scheduler library.

2015-07-01 Thread Mesos ReviewBot
, 35857, 35858, 36079] All tests passed. - Mesos ReviewBot On July 1, 2015, 5:28 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36079

Re: Review Request 36041: The configure phase breaks with the IBM JVM.

2015-06-30 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 30, 2015, 9:09 a.m., Jihun Kang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36041/ --- (Updated

Re: Review Request 36056: Add test cases to check the ownership of persistent volume.

2015-06-30 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 30, 2015, 4:53 p.m., haosdent huang wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36056

Re: Review Request 36036: Add `version` string to MasterInfo

2015-07-01 Thread Mesos ReviewBot
. - Mesos ReviewBot On June 30, 2015, 11:23 p.m., Marco Massenzio wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36036

Re: Review Request 36127: Removing '.json' extension in files endpoints url

2015-07-04 Thread Mesos ReviewBot
. - Mesos ReviewBot On July 2, 2015, 8:03 a.m., Isabel Jimenez wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36127

<    1   2   3   4   5   6   7   8   9   10   >