Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

> On Aug. 5, 2015, 6:05 a.m., Marco Massenzio wrote: > > Ship It! Tested on Mac OSX 10.10 (Python 2.7) and Ubuntu 14.04: by `export PYTHONPATH=$MESOS_INSTALL_DIR/lib/python/site-packages` importing `mesos` and associated modules works. Thanks for fixing this, @haosdent! - Marco --

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/#review94196 --- Ship it! Ship It! - Marco Massenzio On Aug. 5, 2015, 4:21 a.m.,

Re: Review Request 36987: Extended 'getFormValue' to 'getFormValues' in order to parse input uniformly.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36987/ --- (Updated Aug. 5, 2015, 6:05 a.m.) Review request for mesos, Ben Mahler and Jie

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

> On Aug. 5, 2015, 5:47 a.m., Marco Massenzio wrote: > > did you forget to rebase? Never mind - I was just looking at diff 4 --> 5. All good. - Marco --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/#review94191 --- did you forget to rebase? - Marco Massenzio On Aug. 5, 2015, 4:21

Re: Review Request 35702: Added /reserve HTTP endpoint to the master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35702/#review94154 --- Ship it! LGTM overall. Please address the remaining issues and comm

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/ --- (Updated Aug. 5, 2015, 5:25 a.m.) Review request for mesos and Ben Mahler. Bu

Re: Review Request 37105: Removed the code of checkpointing container root filesystem path.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37105/#review94185 --- Patch looks great! Reviews applied: [36929, 36930, 36954, 36956, 37

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

> On Aug. 4, 2015, 5:17 p.m., Marco Massenzio wrote: > > Can you please define how did you test that this works in the 'tests' > > section? > > I can try this out on Ubuntu 14.04 (and OSX) given the steps to reproduce > > the issue, and then applying the patch. > > > > Once we verify that it w

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/ --- (Updated Aug. 5, 2015, 4:21 a.m.) Review request for mesos, Kapil Arya and Marc

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/ --- (Updated Aug. 5, 2015, 4:19 a.m.) Review request for mesos, Kapil Arya and Marc

Re: Review Request 36979: Updating all references to os::shell

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36979/#review94177 --- src/slave/containerizer/isolators/network/port_mapping.cpp (line 15

Re: Review Request 36811: Don't check protobuf jar when --disable-java flag.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36811/#review94176 --- configure.ac (line 544)

Re: Review Request 36810: Don't check protobuf jar in libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36810/#review94175 --- Ship it! Seems reasonable to me. I haven't run a build personally t

Re: Review Request 37101: Remove unused sched API's

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37101/#review94174 --- Patch looks great! Reviews applied: [36410, 36411, 36412, 36413, 37

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34835/#review94172 --- docs/mesos-c++-style-guide.md (line 557)

Re: Review Request 36978: MESOS-3142 Refactoring os::shell - patch 1/2

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36978/#review94162 --- Ship it! 3rdparty/libprocess/3rdparty/stout/include/stout/os/posix

Re: Review Request 37109: Removed ability to mutate user from scheduler library

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37109/#review94171 --- Ship it! Ship It! - Vinod Kone On Aug. 5, 2015, 1:23 a.m., Anand

Re: Review Request 37108: Remove unused failover functionality in scheduler library

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37108/#review94170 --- Ship it! Ship It! - Vinod Kone On Aug. 5, 2015, 1:19 a.m., Anand

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

> On Aug. 4, 2015, 5:17 p.m., Marco Massenzio wrote: > > Can you please define how did you test that this works in the 'tests' > > section? > > I can try this out on Ubuntu 14.04 (and OSX) given the steps to reproduce > > the issue, and then applying the patch. > > > > Once we verify that it w

Re: Review Request 37053: Set TaskStatus.executor_id on receiving a StatusUpdate from Executor.

> On Aug. 4, 2015, 11:03 p.m., Niklas Nielsen wrote: > > src/slave/slave.cpp, lines 2720-2721 > > > > > > What do you think about emitting a warning if the executor id is > > already set? It could be, that users ha

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/#review94165 --- Patch looks great! Reviews applied: [37081] All tests passed. - M

Review Request 37109: Removed ability to mutate user from scheduler library

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37109/ --- Review request for mesos and Vinod Kone. Repository: mesos Description --

Review Request 37108: Remove unused failover functionality in scheduler library

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37108/ --- Review request for mesos, Ben Mahler and Vinod Kone. Repository: mesos Descri

Re: Review Request 36783: Windows: Header splitting continued (stout/os.hpp)

> On Aug. 5, 2015, 12:38 a.m., Ben Mahler wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/os.hpp, lines 22-50 > > > > > > Why do we have both os.hpp and os/os.hpp? Seems strange to put only the > > str

Re: Review Request 37097: Fix 'Accept-Encoding' parsing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37097/#review94161 --- 3rdparty/libprocess/src/http.cpp

Review Request 37106: PortMappingIsolatorProcess shell script can silently fail.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37106/ --- Review request for mesos, Chi Zhang, Ian Downes, Jie Yu, and Cong Wang. Bugs: M

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/#review94156 --- Ship it! src/linux/perf.cpp (lines 292 - 294)

Re: Review Request 36783: Windows: Header splitting continued (stout/os.hpp)

> On Aug. 4, 2015, 5:38 p.m., Ben Mahler wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/os.hpp, lines 22-50 > > > > > > Why do we have both os.hpp and os/os.hpp? Seems strange to put only the > > stru

Re: Review Request 36783: Windows: Header splitting continued (stout/os.hpp)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36783/#review94159 --- 3rdparty/libprocess/3rdparty/stout/include/stout/os/os.hpp (lines 2

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/ --- (Updated Aug. 5, 2015, 12:32 a.m.) Review request for mesos, Anand Mazumdar, Be

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/ --- (Updated Aug. 5, 2015, 12:24 a.m.) Review request for mesos, Anand Mazumdar, Be

Re: Review Request 37082: Tests for subscribe/failover functionality for http based framework

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37082/ --- (Updated Aug. 5, 2015, midnight) Review request for mesos, Ben Mahler and Vinod

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/ --- (Updated Aug. 4, 2015, 11:59 p.m.) Review request for mesos and Ben Mahler. C

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/#review94152 --- Patch looks great! Reviews applied: [37045] All tests passed. - M

Re: Review Request 36720: Add subscribe-> subscribed workflow for http frameworks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36720/ --- (Updated Aug. 4, 2015, 11:57 p.m.) Review request for mesos, Ben Mahler and Vin

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

> On Aug. 4, 2015, 11:45 p.m., Ben Mahler wrote: > > src/linux/perf.cpp, lines 314-316 > > > > > > Any reason you're changing the style of the failure messages? Let's > > leave them untouched in this patch, since th

Review Request 37105: Removed the code of checkpointing container root filesystem path.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37105/ --- Review request for mesos, Ian Downes, Timothy Chen, Vinod Kone, and Jiang Yan Xu

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/#review94148 --- Looks great paul! Just some minor things around style before we can

Re: Review Request 36402: Adding 'Accept' header in request

> On Aug. 4, 2015, 7:25 p.m., Ben Mahler wrote: > > Looking better! Can you please pull out the fixes to 'acceptsEncoding' into > > a separate review? Also, would like to see tests for the cases that weren't > > handled correctly (e.g. "gzipp;q=1.0" should not match "gzip"). Pulling the > > ch

Re: Review Request 37091: Stopped mutating executor info with default container info.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37091/#review94145 --- Ship it! Ship It! - Timothy Chen On Aug. 4, 2015, 7:01 p.m., Jie

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/ --- (Updated Aug. 4, 2015, 11:05 p.m.) Review request for mesos, Anand Mazumdar, Be

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/ --- (Updated Aug. 4, 2015, 11:05 p.m.) Review request for mesos, Anand Mazumdar, Be

Re: Review Request 37053: Set TaskStatus.executor_id on receiving a StatusUpdate from Executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37053/#review94143 --- Ship it! src/slave/slave.cpp (lines 2720 - 2721)

Re: Review Request 36720: Add subscribe-> subscribed workflow for http frameworks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36720/#review94130 --- I'll get the exited changes committed. src/master/master.cpp (line

Re: Review Request 36402: Adding 'Accept' header in request

> On Aug. 4, 2015, 7:25 p.m., Ben Mahler wrote: > > 3rdparty/libprocess/src/http.cpp, line 219 > > > > > > This will crash the program if tokens is empty! > > Isabel Jimenez wrote: > This will not crash, tokeniz

Re: Review Request 37097: Fix 'Accept-Encoding' parsing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37097/ --- (Updated Aug. 4, 2015, 10:43 p.m.) Review request for mesos and Ben Mahler. R

Re: Review Request 36720: Add subscribe-> subscribed workflow for http frameworks

> On Aug. 4, 2015, 9:07 p.m., Ben Mahler wrote: > > I'll get the /call validation committed. Let's follow up to get tests added for each of the error cases in /call. - Ben --- This is an automatically generated e-mail. To reply, visit:

Review Request 37101: Remove unused sched API's

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37101/ --- Review request for mesos, Ian Downes, Jie Yu, and Vinod Kone. Repository: mesos

Re: Review Request 36913: Added /quota HTTP Endpoint for Quota handling.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36913/#review94133 --- Patch looks great! Reviews applied: [36913] All tests passed. - M

Re: Review Request 37082: Tests for subscribe/failover functionality for http based framework

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37082/ --- (Updated Aug. 4, 2015, 10:02 p.m.) Review request for mesos, Ben Mahler and Vin

Re: Review Request 36720: Add subscribe-> subscribed workflow for http frameworks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36720/ --- (Updated Aug. 4, 2015, 10 p.m.) Review request for mesos, Ben Mahler and Vinod

Re: Review Request 37097: Fix 'Accept-Encoding' parsing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37097/ --- (Updated Aug. 4, 2015, 9:29 p.m.) Review request for mesos and Ben Mahler. Re

Re: Review Request 36720: Add subscribe-> subscribed workflow for http frameworks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36720/#review94111 --- I'll get the /call validation committed. src/master/http.cpp (line

Re: Review Request 36910: Patch configure.ac to include $LIBS in the CRAM-MD5 check

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36910/#review94121 --- Patch looks great! Reviews applied: [36910] All tests passed. - M

Re: Review Request 36402: Adding 'Accept' header in request

> On Aug. 4, 2015, 7:25 p.m., Ben Mahler wrote: > > 3rdparty/libprocess/src/http.cpp, line 219 > > > > > > This will crash the program if tokens is empty! > > Isabel Jimenez wrote: > This will not crash, tokeniz

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

> On Aug. 4, 2015, 5:17 p.m., Marco Massenzio wrote: > > Can you please define how did you test that this works in the 'tests' > > section? > > I can try this out on Ubuntu 14.04 (and OSX) given the steps to reproduce > > the issue, and then applying the patch. > > > > Once we verify that it w

Re: Review Request 36402: Adding 'Accept' header in request

> On Aug. 4, 2015, 7:25 p.m., Ben Mahler wrote: > > 3rdparty/libprocess/src/http.cpp, line 219 > > > > > > This will crash the program if tokens is empty! > > Isabel Jimenez wrote: > This will not crash, tokeniz

Re: Review Request 36402: Adding 'Accept' header in request

> On Aug. 4, 2015, 7:25 p.m., Ben Mahler wrote: > > 3rdparty/libprocess/src/http.cpp, line 156 > > > > > > This will crash the program if tokens is empty! Please see comment below. - Isabel -

Re: Review Request 36402: Adding 'Accept' header in request

> On Aug. 4, 2015, 7:25 p.m., Ben Mahler wrote: > > 3rdparty/libprocess/src/http.cpp, line 219 > > > > > > This will crash the program if tokens is empty! This will not crash, tokenize will always return a vector of

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/#review94112 --- 3rdparty/libprocess/src/http.cpp (line 155)

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

> On Aug. 4, 2015, 5:17 p.m., Marco Massenzio wrote: > > Can you please define how did you test that this works in the 'tests' > > section? > > I can try this out on Ubuntu 14.04 (and OSX) given the steps to reproduce > > the issue, and then applying the patch. > > > > Once we verify that it w

Re: Review Request 35797: Updated Frameworkinfo.capabilities on framework re-registration to support adding capabilities

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35797/#review94102 --- Looks like you realized that you need to create a new review :) So I

Re: Review Request 37075: Protobuf definitions instructing the fetcher cache about checksums and their validation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37075/#review94103 --- Patch looks great! Reviews applied: [37075] All tests passed. - M

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/#review94105 --- Looking better! Can you please pull out the fixes to 'acceptsEncodin

Re: Review Request 37072: Added test for allocator update on scheduler failover

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37072/#review94104 --- Nice test! src/tests/oversubscription_tests.cpp (line 864)

Review Request 37091: Stopped mutating executor info with default container info.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37091/ --- Review request for mesos, Ben Mahler, Ian Downes, Timothy Chen, and Vinod Kone.

Re: Review Request 37053: Set TaskStatus.executor_id on receiving a StatusUpdate from Executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37053/ --- (Updated Aug. 4, 2015, 2:32 p.m.) Review request for mesos, Niklas Nielsen and

Review Request 37053: Set TaskStatus.executor_id on receiving a StatusUpdate from Executor.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37053/ --- Review request for mesos, Niklas Nielsen and Vinod Kone. Bugs: MESOS-3196 h

Re: Review Request 36956: Created a test abstraction for preparing test rootfs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36956/#review93921 --- Ship it! src/tests/containerizer/rootfs.hpp (line 66)

Re: Review Request 37012: mesos: Removed unused 'fatal' and 'fatalerror' macros.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37012/ --- (Updated Aug. 4, 2015, 5:43 p.m.) Review request for mesos, Benjamin Hindman, B

Re: Review Request 37011: libprocess: Removed unused 'fatal' and 'fatalerror' macros.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37011/ --- (Updated Aug. 4, 2015, 5:43 p.m.) Review request for mesos, Benjamin Hindman, B

Re: Review Request 36402: Adding 'Accept' header in request

> On Aug. 3, 2015, 9:38 p.m., Ben Mahler wrote: > > Is this ready to review? Mind updating the 'issues' accordingly? Just did, thanks for comments :) - Isabel --- This is an automatically generated e-mail. To reply, visit: https://revie

Re: Review Request 36402: Adding 'Accept' header in request

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36402/ --- (Updated Aug. 4, 2015, 5:42 p.m.) Review request for mesos, Anand Mazumdar, Ben

Re: Review Request 37010: stout: Removed unused 'fatal' and 'fatalerror' macros.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37010/ --- (Updated Aug. 4, 2015, 5:42 p.m.) Review request for mesos, Benjamin Hindman, B

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

> On Aug. 4, 2015, 6:36 a.m., Michael Park wrote: > > src/tests/containerizer/memory_test_helper.cpp, line 81 > > > > > > What's the significance in the order in which these are called? I > > would've expected someth

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

> On Aug. 4, 2015, 5:17 p.m., Marco Massenzio wrote: > > Can you please define how did you test that this works in the 'tests' > > section? > > I can try this out on Ubuntu 14.04 (and OSX) given the steps to reproduce > > the issue, and then applying the patch. > > > > Once we verify that it w

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/ --- (Updated Aug. 4, 2015, 5:33 p.m.) Review request for mesos, Kapil Arya and Marc

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

> On Aug. 4, 2015, 6:36 a.m., Michael Park wrote: > > src/tests/containerizer/memory_test_helper.cpp, line 81 > > > > > > What's the significance in the order in which these are called? I > > would've expected someth

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/#review94084 --- Can you please define how did you test that this works in the 'tests

Re: Review Request 37082: Tests for subscribe/failover functionality for http based framework

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37082/ --- (Updated Aug. 4, 2015, 5:15 p.m.) Review request for mesos, Ben Mahler and Vino

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

> On Aug. 4, 2015, 6:36 a.m., Michael Park wrote: > > src/tests/containerizer/memory_test_helper.cpp, line 81 > > > > > > What's the significance in the order in which these are called? I > > would've expected someth

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/ --- (Updated Aug. 4, 2015, 5:12 p.m.) Review request for mesos, Kapil Arya and Marc

Re: Review Request 37045: Convert Linux perf sampler to use process:await().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37045/ --- (Updated Aug. 4, 2015, 4:57 p.m.) Review request for mesos and Ben Mahler. Ch

Review Request 37082: Tests for subscribe/failover functionality for http based framework

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37082/ --- Review request for mesos, Ben Mahler and Vinod Kone. Repository: mesos Descri

Re: Review Request 36720: Add subscribe-> subscribed workflow for http frameworks

> On July 29, 2015, 7:12 p.m., Ben Mahler wrote: > > src/master/master.cpp, lines 2128-2159 > > > > > > Any reason you're skipping validation (authorization) of the framework? > > Looks like we should pull out the

Re: Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/ --- (Updated Aug. 4, 2015, 4:33 p.m.) Review request for mesos, Kapil Arya and Marc

Re: Review Request 36720: Add subscribe-> subscribed workflow for http frameworks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36720/ --- (Updated Aug. 4, 2015, 4:33 p.m.) Review request for mesos, Ben Mahler and Vino

Review Request 37081: Let __init__.py getting installed to $PREFIX/lib/pythonX.Y/site-packages/mesos.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37081/ --- Review request for mesos and Marco Massenzio. Bugs: MESOS-2337 https://issu

Re: Review Request 36050: Added test authorizer module.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36050/ --- (Updated Aug. 4, 2015, 6:27 p.m.) Review request for mesos, Adam B, Bernd Mathi

Re: Review Request 36050: Added test authorizer module.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36050/ --- (Updated Aug. 4, 2015, 6:27 p.m.) Review request for mesos, Adam B, Bernd Mathi

Re: Review Request 36049: Added support for modularized Authorizer

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36049/ --- (Updated Aug. 4, 2015, 6:26 p.m.) Review request for mesos, Adam B, Bernd Mathi

Review Request 37080: Introduced RecordIO response reader

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37080/ --- Review request for mesos, Ben Mahler and Vinod Kone. Repository: mesos Descri

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

> On Aug. 4, 2015, 6:36 a.m., Michael Park wrote: > > src/tests/containerizer/memory_test_helper.cpp, line 81 > > > > > > What's the significance in the order in which these are called? I > > would've expected someth

Re: Review Request 36867: Add "labels" to FrameworkInfo.

Hi folks, Thanks for the great feedback! I addressed almost all of these issues, but had a few things left to wrap up before uploading a new patch -- I'm traveling at the moment, but I should have a new patch by tomorrow. Neil On Mon, Aug 3, 2015 at 8:45 PM Adam B wrote: > This is an automatical

Re: Review Request 36847: Added HTTP Delete Method.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36847/#review94067 --- Patch looks great! Reviews applied: [36847] All tests passed. - M

Re: Review Request 37065: Fixed MemIsolatorTest failure on OSX.

> On Aug. 3, 2015, 11:36 p.m., Michael Park wrote: > > src/tests/containerizer/memory_test_helper.cpp, line 81 > > > > > > What's the significance in the order in which these are called? I > > would've expected somet

Re: Review Request 36913: Added /quota HTTP Endpoint for Quota handling.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36913/ --- (Updated Aug. 4, 2015, 2:32 p.m.) Review request for mesos, Alexander Rukletsov

Re: Review Request 36048: Updated authorizer to allow for modularized implementations.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36048/#review94059 --- include/mesos/authorizer/authorizer.hpp (line 83)

Re: Review Request 36913: Added /quota HTTP Endpoint for Quota handling.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36913/#review94058 --- Patch looks great! Reviews applied: [36913] All tests passed. - M

  1   2   >