Re: Review Request 41790: Add tests for /weights endpoint.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41790/ --- (Updated Jan. 12, 2016, 7:30 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 41789: Expose the http::internal::request function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41789/ --- (Updated Jan. 12, 2016, 7:30 a.m.) Review request for mesos, Adam B, Joerg Scha

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/ --- (Updated Jan. 12, 2016, 7:29 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 Jan. 12, 2016, 7:29 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 42086: Updated and refactored Master::accept for Offers with InverseOffers.

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

Re: Review Request 41879: Removed constructor which uses restricted width.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41879/ --- (Updated Jan. 12, 2016, 7:57 a.m.) Review request for mesos, Michael Park and J

Re: Review Request 41878: Consistently used fixed width types.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41878/ --- (Updated Jan. 12, 2016, 7:57 a.m.) Review request for mesos, Michael Park and J

Re: Review Request 42130: WIP: Added tests for `ALLOCATION_SLACK` helper functions.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42130/ --- (Updated Jan. 12, 2016, 2:28 p.m.) Review request for mesos, Guangya Liu and Ji

Re: Review Request 42097: Added cgroup/net_cls to the mesos containerizer isolation flags.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42097/ --- (Updated Jan. 12, 2016, 6:28 a.m.) Review request for mesos and Jie Yu. Bugs:

Re: Review Request 42097: Added cgroup/net_cls to the mesos containerizer isolation flags.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42097/ --- (Updated Jan. 12, 2016, 6:28 a.m.) Review request for mesos and Jie Yu. Bugs:

Re: Review Request 42048: Defined of the CgroupNetClsIsolatorProcess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42048/ --- (Updated Jan. 12, 2016, 6:27 a.m.) Review request for mesos and Jie Yu. Bugs:

Re: Review Request 42047: Specified the CgroupsNetClsIsolatorProcess class. This adds the ability to isolate a mesos container using the net_cls cgroup subsystem.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42047/ --- (Updated Jan. 12, 2016, 6:26 a.m.) Review request for mesos and Jie Yu. Bugs:

Review Request 42098: Added unit test-case for CgroupsNetClsIsolatorProcess.

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

Re: Review Request 42053: Add flags to set size of completed task/framework history.

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

Re: Review Request 42172: Add documentation for logging and ContainerLogger.

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

Re: Review Request 41789: Expose the http::internal::request function.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41789/ --- (Updated Jan. 12, 2016, 4:05 a.m.) Review request for mesos, Adam B, Joerg Scha

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/ --- (Updated Jan. 12, 2016, 4:04 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 Jan. 12, 2016, 4:04 a.m.) Review request for mesos, Adam B, Neil Conwa

Re: Review Request 42157: Changed ProvisionerAppcTest to use AppcStoreTest suite.

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

Re: Review Request 41820: Pulled out provisioner from linux filesystem isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41820/#review113923 --- Bad patch! Reviews applied: [41814, 41999, 41815, 41816, 41817, 41

Re: Review Request 42123: Enabled load qos controller use USAGE SLACK revocable resources.

> On Jan. 11, 2016, 11:33 a.m., Klaus Ma wrote: > > Would you add test case for that? It seems `make check` will also pass > > without this patch :). > > Guangya Liu wrote: > I think that we need to update oversubscription_tests.cpp to use > usageSlack() instead of revocable() to get usage

Re: Review Request 42106: Added helper function to return Error in condition.

> On Jan. 12, 2016, 2:03 a.m., Joseph Wu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/errorbase.hpp, line 57 > > > > > > Why is there a redundant `while` here? It's used to avoid un-expected behaviour,

Re: Review Request 42040: Added Quota Operator Documentation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42040/#review113916 --- Ship it! docs/quota.md (line 61)

Re: Review Request 40632: Enabled oversubscribed resources for reservations in allocator.

> On Dec. 30, 2015, 8:42 p.m., Klaus Ma wrote: > > src/master/allocator/mesos/hierarchical.cpp, line 1302 > > > > > > If framework has reserved.statless resources, allocator should not > > dispatch it as ALLOCATION

Re: Review Request 40632: Enabled oversubscribed resources for reservations in allocator.

> On 十二月 30, 2015, 12:42 p.m., Klaus Ma wrote: > > src/master/allocator/mesos/hierarchical.cpp, line 1302 > > > > > > If framework has reserved.statless resources, allocator should not > > dispatch it as ALLOCATION

Re: Review Request 42040: Added Quota Operator Documentation.

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

Re: Review Request 40632: Enabled oversubscribed resources for reservations in allocator.

> On Dec. 30, 2015, 8:42 p.m., Klaus Ma wrote: > > src/master/allocator/mesos/hierarchical.cpp, line 1302 > > > > > > If framework has reserved.statless resources, allocator should not > > dispatch it as ALLOCATION

Re: Review Request 42165: Removed `hasPrincipal` parameter from unreserve validation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42165/#review113913 --- Greg, I think that the document of reservation.md should also be up

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

> On Nov. 4, 2015, 6:08 p.m., Michael Hopcroft wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/stat.hpp, line > > 130 > > > > > > Might want to static_assert that _USE_32BIT_TIME_T is not defi

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39803/ --- (Updated Jan. 12, 2016, 2:03 a.m.) Review request for mesos, Artem Harutyunyan,

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

> On Jan. 11, 2016, 8:45 p.m., Benjamin Bannier wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/stat.hpp, line > > 102 > > > > > > Please do not use a `default` branch when switching over enum

Re: Review Request 41338: Added REASON_RESOURCES_PREEMPTED to mesos proto.

> On Jan. 12, 2016, 9:49 a.m., Joseph Wu wrote: > > Looks like the diff doesn't show up on the WebUI (The downloaded version > > looks fine though). This is possibly because r41434 is private. I've included this code in r41859. @Guangya, would you help to check whether this patch is still nec

Re: Review Request 42086: Updated and refactored Master::accept for Offers with InverseOffers.

> On Jan. 9, 2016, 5:02 p.m., Klaus Ma wrote: > > src/master/master.cpp, line 3078 > > > > > > If both `offerError.isSome()` and `inverseOfferError.isSome()`, should > > we return `Error()`? > > Joseph Wu wrote: >

Re: Review Request 41338: Added REASON_RESOURCES_PREEMPTED to mesos proto.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41338/#review113904 --- Looks like the diff doesn't show up on the WebUI (The downloaded ve

Re: Review Request 41243: Updated how we find the .git directory in bootstrap.

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

Re: Review Request 42053: Add flags to set size of completed task/framework history.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42053/#review113896 --- Looks great, thanks! Just a few trivial comments below, and we can

Re: Review Request 42167: Fixed stout protobuf::parse passing error message in nested JSON.

> On Jan. 12, 2016, 1:24 a.m., Joseph Wu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp, line 336 > > > > > > s/for recursive/for the recursive/ > > s/sequencially/sequentially/ We ende

Re: Review Request 42167: Fixed stout protobuf::parse passing error message in nested JSON.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42167/#review113894 --- 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp (lin

Re: Review Request 42166: Fixed stout protobuf::parse to support JSON containing JSON::Null.

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

Re: Review Request 42167: Fixed stout protobuf::parse passing error message in nested JSON.

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

Re: Review Request 41943: Added test case for stout protobuf parse containing JSON null.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41943/#review113883 --- Ship it! 3rdparty/libprocess/3rdparty/stout/tests/protobuf_tests.

Re: Review Request 42058: Added test case for protobuf parse passing error msg form nest JSON.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42058/#review113889 --- Ship it! 3rdparty/libprocess/3rdparty/stout/tests/protobuf_tests.

Re: Review Request 42154: Moved push_back to end of removeFramework.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42154/#review113893 --- Ship it! Thank you! - Ben Mahler On Jan. 11, 2016, 9:41 p.m., K

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

> On Nov. 4, 2015, 6:08 p.m., Michael Hopcroft wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/stat.hpp, line > > 130 > > > > > > Might want to static_assert that _USE_32BIT_TIME_T is not defi

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

> On Nov. 4, 2015, 6:08 p.m., Michael Hopcroft wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/stat.hpp, line > > 130 > > > > > > Might want to static_assert that _USE_32BIT_TIME_T is not defi

Review Request 42172: Add documentation for logging and ContainerLogger.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42172/ --- Review request for mesos, Benjamin Hindman, Artem Harutyunyan, and Neil Conway.

Re: Review Request 42164: Allowed (un)reserve operations without a principal.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42164/ --- (Updated Jan. 12, 2016, 12:52 a.m.) Review request for mesos, Alexander Rojas,

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

> On Jan. 11, 2016, 8:45 p.m., Benjamin Bannier wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/stat.hpp, line > > 102 > > > > > > Please do not use a `default` branch when switching over enum

Re: Review Request 39802: Windows: Implemented `stout/os/windows/ls.hpp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39802/ --- (Updated Jan. 12, 2016, 12:46 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 41959: Statically initializing fetcher plugins.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41959/ --- (Updated Jan. 12, 2016, 12:42 a.m.) Review request for mesos and Jie Yu. Chan

Re: Review Request 42157: Changed ProvisionerAppcTest to use AppcStoreTest suite.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42157/ --- (Updated Jan. 12, 2016, 12:42 a.m.) Review request for mesos and Jie Yu. Chan

Re: Review Request 39802: Windows: Implemented `stout/os/windows/ls.hpp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39802/ --- (Updated Jan. 12, 2016, 12:40 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 39802: Windows: Implemented `stout/os/windows/ls.hpp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39802/ --- (Updated Jan. 12, 2016, 12:39 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 42157: Changed ProvisionerAppcTest to use AppcStoreTest suite.

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

Re: Review Request 42096: Fixed race in persistent volume tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42096/#review113878 --- Ship it! Ship It! - Michael Park On Jan. 9, 2016, 1:19 a.m., Gr

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39019/ --- (Updated Jan. 12, 2016, 12:24 a.m.) Review request for mesos, Alex Naparu, Dani

Re: Review Request 42153: mesos: Fixed typos in comments and docs.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42153/#review113872 --- Ship it! Ship It! - Joris Van Remoortere On Jan. 11, 2016, 6:29

Re: Review Request 42152: libprocess: Fixed typos in comments.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42152/#review113871 --- Ship it! Ship It! - Joris Van Remoortere On Jan. 11, 2016, 6:29

Re: Review Request 42151: stout: Fixed typo in warning message.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42151/#review113870 --- Ship it! Ship It! - Joris Van Remoortere On Jan. 11, 2016, 6:28

Re: Review Request 42058: Added test case for protobuf parse passing error msg form nest JSON.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42058/ --- (Updated Jan. 11, 2016, 3:40 p.m.) Review request for mesos, Ben Mahler, Artem

Re: Review Request 41943: Added test case for stout protobuf parse containing JSON null.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41943/ --- (Updated Jan. 11, 2016, 3:40 p.m.) Review request for mesos, Ben Mahler, Artem

Re: Review Request 42058: Fixed stout protobuf::parse by passing error message in nested JSON.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42058/ --- (Updated Jan. 11, 2016, 3:39 p.m.) Review request for mesos, Ben Mahler, Artem

Review Request 42167: Fixed stout protobuf::parse passing error message in nested JSON.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42167/ --- Review request for mesos, Ben Mahler, Artem Harutyunyan, Jie Yu, Joris Van Remoo

Re: Review Request 41943: Fixed stout protobuf::parse to support JSON containing JSON::Null.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41943/ --- (Updated Jan. 11, 2016, 3:39 p.m.) Review request for mesos, Ben Mahler, Artem

Review Request 42166: Fixed stout protobuf::parse to support JSON containing JSON::Null.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42166/ --- Review request for mesos, Ben Mahler, Artem Harutyunyan, Jie Yu, Joris Van Remoo

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39019/ --- (Updated Jan. 11, 2016, 11:12 p.m.) Review request for mesos, Alex Naparu, Dani

Review Request 42164: Allowed (un)reserve operations without a principal.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42164/ --- Review request for mesos, Alexander Rojas, Michael Park, and Neil Conway. Bugs:

Re: Review Request 42096: Fixed race in persistent volume tests.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42096/#review113864 --- Ship it! Ship It! - Jie Yu On Jan. 9, 2016, 1:19 a.m., Greg Man

Review Request 42165: Removed `hasPrincipal` parameter from unreserve validation.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42165/ --- Review request for mesos, Alexander Rojas, Michael Park, and Neil Conway. Bugs:

Re: Review Request 39780: Update OversubscriptionTest to not assume dynamic dlopen search.

> On Nov. 30, 2015, 8:51 a.m., Benjamin Bannier wrote: > > src/tests/oversubscription_tests.cpp, line 117 > > > > > > This isn't really too nice, and e.g. cmake does not put build libraries > > into these paths (i.e

Re: Review Request 39782: Add a comment for os::libraries::setPaths.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39782/ --- (Updated Jan. 11, 2016, 10:32 p.m.) Review request for mesos, Benjamin Bannier,

Re: Review Request 39781: Update ModuleTest to not assume dynamic dlopen search.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39781/ --- (Updated Jan. 11, 2016, 10:32 p.m.) Review request for mesos, Benjamin Bannier,

Re: Review Request 39780: Update OversubscriptionTest to not assume dynamic dlopen search.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39780/ --- (Updated Jan. 11, 2016, 10:32 p.m.) Review request for mesos, Benjamin Bannier,

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39019/ --- (Updated Jan. 11, 2016, 10:19 p.m.) Review request for mesos, Alex Naparu, Dani

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

> On Jan. 9, 2016, 12:42 a.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/internal/windows/dirent.hpp, > > line 204 > > > > > > why assert vs. false with errno? > > Alex Clemm

Re: Review Request 39802: Windows: Implemented `stout/os/windows/ls.hpp`.

> On Jan. 11, 2016, 12:38 a.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/ls.hpp, lines > > 36-42 > > > > > > If this is the only section that is different between th

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39019/ --- (Updated Jan. 11, 2016, 10:03 p.m.) Review request for mesos, Alex Naparu, Dani

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

> On Jan. 11, 2016, 9:54 p.m., Joris Van Remoortere wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/internal/windows/dirent.hpp, > > line 63 > > > > > > `>=` if `MAX_PATH` is supposed to include the n

Re: Review Request 39019: Windows: Added dirent compat code for non-Unix systems.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39019/#review113853 --- 3rdparty/libprocess/3rdparty/stout/include/stout/internal/windows/

Re: Review Request 41459: Invoked `_Deferred`'s `operator F()` explicitly.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41459/#review113852 --- Ship it! Ship It! - Daniel Pravat On Jan. 5, 2016, 11:58 p.m.,

Review Request 42154: Moved push_back to end of removeFramework.

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

Re: Review Request 42053: Add flags to set size of completed task/framework history.

- Kevin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42053/#review113428 --- On Jan. 11, 2016, 9:39 p.m., Kevin Klues wrote: > > ---

Re: Review Request 42053: Add flags to set size of completed task/framework history.

- Kevin --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42053/#review113428 --- On Jan. 11, 2016, 9:39 p.m., Kevin Klues wrote: > > ---

Re: Review Request 42053: Add flags to set size of completed task/framework history.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42053/ --- (Updated Jan. 11, 2016, 9:39 p.m.) Review request for mesos and Ben Mahler. B

Re: Review Request 42053: Add flags to set size of completed task/framework history.

> On Jan. 8, 2016, 7:16 a.m., Anand Mazumdar wrote: > > LGTM. Two minor issues: > > > > 1. May be we can escape without passing the `Master` pointer around? > > 2. Would be great to have a test to go with this? I am submitting the test as a separate patch. > On Jan. 8, 2016, 7:16 a.m., Anand

Re: Review Request 42086: Updated and refactored Master::accept for Offers with InverseOffers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42086/ --- (Updated Jan. 11, 2016, 1:31 p.m.) Review request for mesos, Ben Mahler, Artem

Re: Review Request 42086: Updated and refactored Master::accept for Offers with InverseOffers.

> On Jan. 8, 2016, 7:13 p.m., Guangya Liu wrote: > > src/master/validation.cpp, lines 678-709 > > > > > > I saw that there is no test cases for offer validation, do you want to > > add inverseOffer test here? I thi

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39803/#review113835 --- 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/stat.h

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

> On Nov. 4, 2015, 1:02 a.m., Michael Hopcroft wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/os/windows/stat.hpp, line > > 40 > > > > > > Might add a comment explaining that when ::stat() returns a valu

Re: Review Request 39803: Windows: Implemented stout/os/stat.hpp`.

> On Nov. 4, 2015, 1:02 a.m., Michael Hopcroft wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/internal/windows/symlink.hpp, > > line 61 > > > > > > Recommend opening a work item for Windows team to add t

Re: Review Request 42149: Replaced mutex in HTTP server for fetcher cache tests with latch.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42149/ --- (Updated Jan. 11, 2016, 12:21 p.m.) Review request for mesos, Alexander Rojas,

Re: Review Request 41820: Pulled out provisioner from linux filesystem isolator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41820/ --- (Updated Jan. 11, 2016, 12:15 p.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 42149: Replaced mutex in HTTP server for fetcher cache tests with latch. Inlined awaiting fetch contention.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42149/#review113829 --- Bad patch! Reviews applied: [42149] Failed command: ./support/app

Re: Review Request 41283: Introduced an Executor Library based on the new executor HTTP API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41283/ --- (Updated Jan. 11, 2016, 8:13 p.m.) Review request for mesos, Ben Mahler and Vin

Re: Review Request 41814: Unified Container: Created ContainerConfig protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41814/ --- (Updated Jan. 11, 2016, 12:13 p.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 41243: Updated how we find the .git directory in bootstrap.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41243/ --- (Updated Jan. 11, 2016, 8:12 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 42121: Replaced `Clock::settle()` with `os::sleep` in test case.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42121/#review113827 --- 3rdparty/libprocess/src/tests/metrics_tests.cpp (lines 280 - 283)

Re: Review Request 41108: CMake: Add sasl and dl link flags, add curl link library and add protobuf library directory.

> On Jan. 8, 2016, 4:43 p.m., Mesos ReviewBot wrote: > > Bad patch! > > > > Reviews applied: [41096, 41185, 41108] > > > > Failed command: ./support/apply-review.sh -n -r 41108 > > > > Error: > > 2016-01-08 16:43:43 URL:https://reviews.apache.org/r/41108/diff/raw/ > > [4052/4052] -> "41108.p

Review Request 42157: Changed ProvisionerAppcTest to use AppcStoreTest suite.

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

Review Request 42156: Refactored up the common image creation in appc tests.

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

  1   2   >