Re: Review Request 38011: Maintenance Primitives: Use the parse> helper instead of a plural MachineID protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38011/#review99190 --- src/common/protobuf_utils.hpp (line 115)

Re: Review Request 37275: [2/2]Generate make batch file to build project in windows.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37275/#review99188 --- Patch looks great! Reviews applied: [37273, 37275] All tests passe

Re: Review Request 37714: Updated Multimap and multihashmap so their signatures resemble that of hashmap and hashset.

> On Sept. 10, 2015, 1:46 a.m., Michael Park wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/multihashmap.hpp, lines > > 96-97 > > > > > > `s/std::make_pair(key, value)/{key, value}/` I wasn't sure if we

Re: Review Request 37714: Updated Multimap and multihashmap so their signatures resemble that of hashmap and hashset.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37714/ --- (Updated Sept. 16, 2015, 8:14 a.m.) Review request for mesos, Joerg Schad, Mich

Re: Review Request 38417: Unified the implementations of image provisioners.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38417/#review99181 --- Ship it! Ship It! - Timothy Chen On Sept. 16, 2015, 1:28 a.m., J

Re: Review Request 38124: Add V1 Support for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38124/ --- (Updated 九月 16, 2015, 6:01 a.m.) Review request for mesos, Alexander Rukletsov,

Re: Review Request 37532: Add QUIESCE call interface to the scheduler

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37532/ --- (Updated 九月 16, 2015, 5:51 a.m.) Review request for mesos and Vinod Kone. Bug

Re: Review Request 38119: Add metrics of messages_quiesce_offers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38119/ --- (Updated 九月 16, 2015, 5:50 a.m.) Review request for mesos, Alexander Rukletsov,

Re: Review Request 37873: Add quiesce logic in allocator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37873/ --- (Updated 九月 16, 2015, 5:50 a.m.) Review request for mesos, Alexander Rukletsov,

Re: Review Request 38120: Add Java Support for QuiesceOffers

> On 九月 15, 2015, 7:12 p.m., Vinod Kone wrote: > > > > Vinod Kone wrote: > can you also update the java test framework to test this call? I see that the reviveOffers also do not have a example, I want to handle those two APIs together in another patch, make sense? Thanks. - Guangya ---

Re: Review Request 38121: Add Python Support for QuiesceOffers

> On 九月 15, 2015, 7:14 p.m., Vinod Kone wrote: > > Update the python test framework to send this call. I see that the reviveOffers also do not have a test and I want to handle those two APIs in another patch, make sense? Thanks. - Guangya

Re: Review Request 38120: Add Java Support for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38120/ --- (Updated 九月 16, 2015, 5:50 a.m.) Review request for mesos, Alexander Rukletsov,

Re: Review Request 38121: Add Python Support for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38121/ --- (Updated 九月 16, 2015, 5:49 a.m.) Review request for mesos, Alexander Rukletsov,

Re: Review Request 38124: Add V1 Support for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38124/ --- (Updated 九月 16, 2015, 5:49 a.m.) Review request for mesos, Alexander Rukletsov,

Re: Review Request 38126: Add UT for QuiesceOffers

> On 九月 15, 2015, 7:43 p.m., Vinod Kone wrote: > > you should've merged this test review with the dependent review. i really > > like tests to be present in the same patch as the code change, to give me > > confidence that the code changes are correct. > > > > also, no tests for the scheduler

Re: Review Request 38126: Add UT for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38126/ --- (Updated 九月 16, 2015, 5:44 a.m.) Review request for mesos, Alexander Rukletsov,

Re: Review Request 38416: Allow HTTP response codes to be checked with code.

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

Re: Review Request 38416: Allow HTTP response codes to be checked with code.

> On Sept. 16, 2015, 1:43 a.m., Guangya Liu wrote: > > it looks like the structs are ordered by response code, so just moving this to the right place assuming so - Timothy --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 38417: Unified the implementations of image provisioners.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38417/#review99163 --- Patch looks great! Reviews applied: [38407, 38408, 38417] All test

Re: Review Request 37275: [2/2]Generate make batch file to build project in windows.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37275/ --- (Updated Sept. 16, 2015, 3:51 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 37273: [1/2]Add CMake macro VsBuildCommand in libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37273/ --- (Updated Sept. 16, 2015, 3:50 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 37273: [1/2]Add CMake macro VsBuildCommand in libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37273/ --- (Updated Sept. 16, 2015, 2:55 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 37275: [2/2]Generate make batch file to build project in windows.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37275/ --- (Updated Sept. 16, 2015, 2:55 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 38143: Adding executor HTTP API protobuf to V1

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38143/#review99156 --- Patch looks great! Reviews applied: [38278, 38143] All tests passe

Re: Review Request 37532: Add QUIESCE call interface to the scheduler

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37532/#review99153 --- Ship it! Ship It! - Yong Qiao Wang On Sept. 15, 2015, 11 p.m., G

Re: Review Request 37873: Add quiesce logic in allocator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37873/#review99152 --- Ship it! Ship It! - Yong Qiao Wang On Sept. 6, 2015, 2:40 a.m.,

Re: Review Request 37273: [1/2]Add CMake macro VsBuildCommand in libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37273/ --- (Updated Sept. 16, 2015, 2:15 a.m.) Review request for mesos, Artem Harutyunyan

Re: Review Request 37273: [1/2]Add CMake macro VsBuildCommand in libprocess.

> On Sept. 15, 2015, 11:12 p.m., Joseph Wu wrote: > > 3rdparty/libprocess/3rdparty/CMakeLists.txt, lines 223-225 > > > > > > Just curious, why aren't you using the protobuf-2.5.0 tarball that's > > included in the

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38367/ --- (Updated Sept. 15, 2015, 10:05 p.m.) Review request for mesos, Connor Doyle, Ji

Re: Review Request 38368: Replaced slaveTaskStatusLabelDecorator hook with slaveTaskStatusDecorator hook.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38368/ --- (Updated Sept. 15, 2015, 10:05 p.m.) Review request for mesos, Connor Doyle, Ji

Re: Review Request 38370: Updated docker executor to set container IP in TaskStatus::NetworkInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38370/#review99149 --- Bad patch! Reviews applied: [38363, 38364, 38365] Failed command:

Re: Review Request 38416: Allow HTTP response codes to be checked with code.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38416/#review99147 --- 3rdparty/libprocess/include/process/http.hpp (lines 559 - 584)

Re: Review Request 37903: stout: Fix bug in IPNetwork::create() with zero prefix.

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

Re: Review Request 38417: Unified the implementations of image provisioners.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38417/ --- (Updated Sept. 16, 2015, 1:28 a.m.) Review request for mesos, Timothy Chen and

Re: Review Request 38417: Unified the implementations of image provisioners.

> On Sept. 16, 2015, 1:14 a.m., Timothy Chen wrote: > > src/slave/flags.cpp, line 62 > > > > > > image types sounds very generic, but I guess could be ok. > > How about image_providers or image_formats? > > No

Re: Review Request 38417: Unified the implementations of image provisioners.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38417/#review99142 --- src/slave/flags.cpp (line 62)

Review Request 38417: Unified the implementations of image provisioners.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38417/ --- Review request for mesos, Timothy Chen and Jiang Yan Xu. Bugs: MESOS-3432 h

Review Request 38416: Allow HTTP response codes to be checked with code.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38416/ --- Review request for mesos, Anand Mazumdar, Jie Yu, Jojy Varghese, and Jiang Yan X

Re: Review Request 38158: Refactored Value::Ranges coalesce().

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

Re: Review Request 37273: [1/2]Add CMake macro VsBuildCommand in libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37273/#review99115 --- 3rdparty/libprocess/3rdparty/CMakeLists.txt (lines 223 - 225)

Re: Review Request 37532: Add QUIESCE call interface to the scheduler

> On 九月 15, 2015, 7:01 p.m., Vinod Kone wrote: > > include/mesos/scheduler.hpp, line 272 > > > > > > Should we even bother adding this to the old API? Note that adding a > > new call to the driver will impose a upg

Re: Review Request 37532: Add QUIESCE call interface to the scheduler

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37532/ --- (Updated 九月 15, 2015, 11 p.m.) Review request for mesos and Vinod Kone. Bugs:

Re: Review Request 37532: Add QUIESCE call interface to the scheduler

> On 九月 15, 2015, 7:01 p.m., Vinod Kone wrote: > > include/mesos/scheduler.hpp, line 272 > > > > > > Should we even bother adding this to the old API? Note that adding a > > new call to the driver will impose a upg

Re: Review Request 38143: Adding executor HTTP API protobuf to V1

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38143/ --- (Updated Sept. 15, 2015, 10:46 p.m.) Review request for mesos, Adam B, Anand Ma

Re: Review Request 38143: Adding executor HTTP API protobuf to V1

> On Sept. 15, 2015, 10:40 p.m., Anand Mazumdar wrote: > > LGTM minus Two Nits: > > > > 1. Can we update the description of the review to just say that this change > > just copies the existing unversioned protobuf to the V1 namespace. > > 2. I am assuming that you would take care of Vinod's ear

Re: Review Request 38143: Adding executor HTTP API protobuf to V1

- Isabel --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38143/#review99117 --- On Sept. 15, 2015, 10:46 p.m., Isabel Jimenez wrote: >

Re: Review Request 38143: Adding executor HTTP API protobuf to V1

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38143/#review99117 --- Ship it! LGTM minus Two Nits: 1. Can we update the description of

Re: Review Request 38143: Adding executor HTTP API protobuf to V1

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38143/ --- (Updated Sept. 15, 2015, 10:35 p.m.) Review request for mesos, Adam B, Anand Ma

Re: Review Request 37275: [2/2]Generate make batch file to build project in windows.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37275/#review99116 --- Patch looks great! Reviews applied: [37273, 37275] All tests passe

Re: Review Request 37540: Add perf event API

> On Sept. 15, 2015, 12:31 a.m., Vinod Kone wrote: > > my main question is regarding division of responsibilities between the > > wrapper class and the process class. Looks like opening the fd and mmaping > > is done by the wrapper and the rest by the process. Can everything be done > > by the

Re: Review Request 38003: MESOS-3351 (duplicated slave id in master after master failover)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38003/#review99112 --- src/tests/master_tests.cpp (line 3596)

Re: Review Request 38278: Updating unversioned Executor protobuf

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38278/#review99114 --- Ship it! Ship It! - Anand Mazumdar On Sept. 15, 2015, 10:24 p.m.

Re: Review Request 38278: Updating unversioned Executor protobuf

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38278/ --- (Updated Sept. 15, 2015, 10:24 p.m.) Review request for mesos, Adam B, Anand Ma

Re: Review Request 38278: Updating unversioned Executor protobuf

> On Sept. 15, 2015, 10:10 p.m., Anand Mazumdar wrote: > > include/mesos/executor/executor.proto, line 163 > > > > > > Minor: Can we also have a comment before the `framework_id` field ? > > > > // Identifie

Re: Review Request 38278: Updating unversioned Executor protobuf

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38278/#review99109 --- Looks good, just a few more minor things. Also to save effort don't

Re: Review Request 38278: Updating unversioned Executor protobuf

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38278/#review99110 --- Ship it! Ship It! - Vinod Kone On Sept. 15, 2015, 9:32 p.m., Isa

Re: Review Request 38011: Maintenance Primitives: Use the parse> helper instead of a plural MachineID protobuf.

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

Re: Review Request 38278: Updating unversioned Executor protobuf

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38278/ --- (Updated Sept. 15, 2015, 9:32 p.m.) Review request for mesos, Adam B, Anand Maz

Re: Review Request 38278: Updating unversioned Executor protobuf

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38278/#review99102 --- include/mesos/executor/executor.proto (line 131)

Re: Review Request 38370: Updated docker executor to set container IP in TaskStatus::NetworkInfo.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38370/ --- (Updated Sept. 15, 2015, 5:11 p.m.) Review request for mesos, Connor Doyle, Jie

Re: Review Request 38368: Replaced slaveTaskStatusLabelDecorator hook with slaveTaskStatusDecorator hook.

> On Sept. 14, 2015, 6:11 p.m., Niklas Nielsen wrote: > > src/examples/test_hook_module.cpp, line 198 > > > > > > How about setting the protocol (for testing)? So maybe have two network > > infos (one with each)? A

Re: Review Request 37903: stout: Fix bug in IPNetwork::create() with zero prefix.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37903/ --- (Updated Sept. 15, 2015, 9:10 p.m.) Review request for mesos. Bugs: MESOS-332

Re: Review Request 38368: Replaced slaveTaskStatusLabelDecorator hook with slaveTaskStatusDecorator hook.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38368/ --- (Updated Sept. 15, 2015, 5:10 p.m.) Review request for mesos, Connor Doyle, Jie

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

> On Sept. 14, 2015, 6:43 p.m., Niklas Nielsen wrote: > > src/slave/slave.cpp, line 2760 > > > > > > s/the container IP address if/the container IP address with the IP from > > the agent PID, if/ or something that

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

> On Sept. 14, 2015, 6:24 p.m., Niklas Nielsen wrote: > > include/mesos/mesos.proto, line 1163 > > > > > > We still go by 70 col comments, right? If so, let's wrap the comment > > blocks. It's been updated :-) ht

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

> On Sept. 14, 2015, 6:19 p.m., Vinod Kone wrote: > > src/common/http.cpp, lines 153-194 > > > > > > Why custom models? Can we not use the protobuf to json converters? This is mostly due to Labels. The way they are

Re: Review Request 38367: Added NetworkInfo message to ContainerInfo and TaskStatus.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38367/ --- (Updated Sept. 15, 2015, 5:06 p.m.) Review request for mesos, Connor Doyle, Jie

Re: Review Request 38366: Added helper to model Labels message for state.json.

> On Sept. 15, 2015, 1:56 p.m., Niklas Nielsen wrote: > > src/common/protobuf_utils.cpp, line 140 > > > > > > This could break existing 3rd party parsing; why not leave it set? I am not so sure. This is fairly incon

Re: Review Request 38365: Updated Isolator::prepare to return list of required namespaces.

> On Sept. 14, 2015, 6:50 p.m., Niklas Nielsen wrote: > > include/mesos/slave/isolator.proto, line 75 > > > > > > Taken we still wrap comments at 70, this needs to be wrapped :) We don't :-). See https://github.com/

Re: Review Request 38365: Updated Isolator::prepare to return list of required namespaces.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38365/ --- (Updated Sept. 15, 2015, 5:05 p.m.) Review request for mesos, Connor Doyle, Jie

Re: Review Request 38364: Minor refactor for MesosContainerizer.

> On Sept. 14, 2015, 6:55 p.m., Niklas Nielsen wrote: > > src/slave/containerizer/mesos/containerizer.cpp, lines 832-843 > > > > > > How about moving this up as well? Fixed. - Kapil -

Re: Review Request 38364: Minor refactor for MesosContainerizer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38364/ --- (Updated Sept. 15, 2015, 5:02 p.m.) Review request for mesos, Connor Doyle, Jie

Re: Review Request 38363: Refactored container Launcher to accept namespaces dynamically.

> On Sept. 14, 2015, 6:32 p.m., Niklas Nielsen wrote: > > src/tests/containerizer/isolator_tests.cpp, line 507 > > > > > > Have you verified the status (non-error) of namespaces() before you get > > here? The Isola

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38408/#review99087 --- src/tests/paths_tests.cpp (line 224)

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

> On Sept. 15, 2015, 8:51 p.m., Timothy Chen wrote: > > Ship It! Wierd, anyway fix it and ship it :) - Timothy --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38408/#review99088 -

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38408/#review99088 --- Ship it! Ship It! - Timothy Chen On Sept. 15, 2015, 6:51 p.m., J

Re: Review Request 38158: Refactored Value::Ranges coalesce().

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38158/ --- (Updated Sept. 15, 2015, 8:46 p.m.) Review request for mesos, Bernd Mathiske an

Re: Review Request 38386: Fix 'operator=' spacing in the registry_client.hpp

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38386/#review99086 --- Ship it! Ship It! src/slave/containerizer/provisioners/docker/reg

Re: Review Request 38386: Fix 'operator=' spacing in the registry_client.hpp

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

Re: Review Request 38143: Adding executor HTTP API protobuf to V1

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38143/ --- (Updated Sept. 15, 2015, 8:16 p.m.) Review request for mesos, Adam B, Anand Maz

Re: Review Request 38278: Updating unversioned Executor protobuf

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38278/ --- (Updated Sept. 15, 2015, 8:13 p.m.) Review request for mesos, Adam B, Anand Maz

Re: Review Request 37969: Maintenance primitives: Tweak validation error messages to return JSON rather than protobuf.

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

Re: Review Request 38126: Add UT for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38126/#review99078 --- you should've merged this test review with the dependent review. i r

Re: Review Request 37873: Add quiesce logic in allocator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37873/#review99077 --- src/master/allocator/mesos/hierarchical.hpp (line 229)

Re: Review Request 38124: Add V1 Support for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38124/#review99076 --- Ship it! Ship It! - Vinod Kone On Sept. 4, 2015, 12:06 p.m., Gua

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

> On Sept. 15, 2015, 7:01 p.m., Jiang Yan Xu wrote: > > src/slave/paths.cpp, line 378 > > > > > > I don't think this needs to be changed to singular and the convention > > is that the plural form ("slaves", "framewo

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

> On Sept. 15, 2015, 12:01 p.m., Jiang Yan Xu wrote: > > src/slave/paths.cpp, line 378 > > > > > > I don't think this needs to be changed to singular and the convention > > is that the plural form ("slaves", "framew

Re: Review Request 38121: Add Python Support for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38121/#review99073 --- Update the python test framework to send this call. src/python/int

Re: Review Request 38120: Add Java Support for QuiesceOffers

> On Sept. 15, 2015, 7:12 p.m., Vinod Kone wrote: > > can you also update the java test framework to test this call? - Vinod --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38120/#review99071

Re: Review Request 38120: Add Java Support for QuiesceOffers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38120/#review99071 --- Ship it! src/java/src/org/apache/mesos/SchedulerDriver.java (lines

Re: Review Request 38119: Add metrics of messages_quiesce_offers

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38119/#review99070 --- Ship it! Ship It! - Vinod Kone On Sept. 4, 2015, 12:06 p.m., Gua

Re: Review Request 37873: Add quiesce logic in allocator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37873/#review99069 --- Ship it! Ship It! - Vinod Kone On Sept. 6, 2015, 2:40 a.m., Guan

Re: Review Request 37873: Add quiesce logic in allocator

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37873/#review99066 --- include/mesos/master/allocator.hpp (line 145)

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

> On Sept. 15, 2015, 7:01 p.m., Jiang Yan Xu wrote: > > src/slave/paths.cpp, line 378 > > > > > > I don't think this needs to be changed to singular and the convention > > is that the plural form ("slaves", "framewo

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38408/#review99065 --- src/slave/containerizer/provisioner/paths.hpp (line 32)

Re: Review Request 37532: Add QUIESCE call interface to the scheduler

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37532/#review99061 --- include/mesos/scheduler.hpp (line 272)

Re: Review Request 38408: Renamed provisioners::path to provisioner::path.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38408/ --- (Updated Sept. 15, 2015, 6:51 p.m.) Review request for mesos, Timothy Chen and

Re: Review Request 38407: Moved files to prepare for unifying provisioners.

> On Sept. 15, 2015, 6:38 p.m., Jiang Yan Xu wrote: > > My only concern is whether this should be checked in before > > https://reviews.apache.org/r/38137/ as it's already in the review process > > for a long time. As long as Tim's OK with it. > > Jie Yu wrote: > r38137 has to fix problems

Re: Review Request 38407: Moved files to prepare for unifying provisioners.

> On Sept. 15, 2015, 11:38 a.m., Jiang Yan Xu wrote: > > My only concern is whether this should be checked in before > > https://reviews.apache.org/r/38137/ as it's already in the review process > > for a long time. As long as Tim's OK with it. > > Jie Yu wrote: > r38137 has to fix problem

Re: Review Request 37273: [1/2]Add CMake macro VsBuildCommand in libprocess.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37273/ --- (Updated Sept. 15, 2015, 6:42 p.m.) Review request for mesos, Artem Harutyunyan

  1   2   >