Re: Review Request 37168: MESOS-3063

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/#review97423 --- As this is an example framework (which will be used by people to sta

Review Request 38046: Replace hard-coded reap interval with a constant

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38046/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-1935 https://issues.ap

Re: Review Request 37989: Replace hard-coded reap interval with a constant

> On 九月 2, 2015, 12:47 a.m., Ben Mahler wrote: > > Hey Guangya, the point of MESOS-1935 was to use these constant functions > > throughout the code base: > > > > ``` > > ? mesos git:(master) ? grep -R 'Seconds(1)' src/tests > > src/tests/containerizer/cgroups_tests.cpp:if (elapsed > Second

Re: Review Request 38046: Replace hard-coded reap interval with a constant

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

Re: Review Request 37280: Maintenance Primitives: Added updateInverseOffer to Allocator.

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

Review Request 38050: Master should send PingSlaveMessage instead of "PING"; Slave should accept PingSlaveMessage but not "PING" message;

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38050/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-1831 and MESOS-1832 ht

Re: Review Request 37826: Introduced conversion of JSON arrays to repeated protobufs.

> On Sept. 1, 2015, 4:46 p.m., Joseph Wu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp, lines 553-555 > > > > > > Can you explain why this is a struct rather than a function? It's because

Re: Review Request 38050: Master should send PingSlaveMessage instead of "PING"; Slave should accept PingSlaveMessage but not "PING" message;

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38050/#review97434 --- src/master/master.cpp

Re: Review Request 37826: Introduced conversion of JSON arrays to repeated protobufs.

> On Sept. 1, 2015, 4:46 p.m., Joseph Wu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp, lines 553-555 > > > > > > Can you explain why this is a struct rather than a function? > > Alexander

Re: Review Request 37168: MESOS-3063

> On Sept. 2, 2015, 7:30 a.m., Joerg Schad wrote: > > As this is an example framework (which will be used by people to start > > their own framework), would it make sense to add some more comments > > explaining especially the relevant bits here? Agree, let me add more comments for the detail

Review Request 38051: Only update the task status when its old status is not terminal.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38051/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-2864 https://issues.ap

Re: Review Request 36908: Added QuotaInfo Protobuf.

> On Aug. 25, 2015, 5:51 a.m., Guangya Liu wrote: > > include/mesos/master/quota.proto, line 19 > > > > > > Yes, does v1 API will be supportted for quota? > > Alexander Rukletsov wrote: > Adding new protobufs sho

Re: Review Request 36908: Added QuotaInfo Protobuf.

> On Aug. 24, 2015, 9:12 p.m., Alexander Rukletsov wrote: > > include/mesos/master/quota.proto, line 19 > > > > > > I think we should version this protobuf. As so far only mesos and scheduler are versioned, we don't

Re: Review Request 36908: Added QuotaInfo Protobuf.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36908/ --- (Updated Sept. 2, 2015, 1:38 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 38051: Only update the task status when its old status is not terminal.

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

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 Sept. 2, 2015, 1:46 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 38050: Master should send PingSlaveMessage instead of "PING"; Slave should accept PingSlaveMessage but not "PING" message;

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

Re: Review Request 38050: Master should send PingSlaveMessage instead of "PING"; Slave should accept PingSlaveMessage but not "PING" message;

> On 九月 2, 2015, 12:55 p.m., Klaus Ma wrote: > > Thanks @klaus Ma for your comments. I have addressed all of them. - Yong Qiao --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38050/#review974

Re: Review Request 37669: Ignore overflow docker components in version parsing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37669/#review97452 --- Ship it! Ship It! - Yong Qiao Wang On 八月 30, 2015, 8:45 a.m., ha

Review Request 38056: Fix webui task informations not update bug.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38056/ --- Review request for mesos, Adam B and Ross Allen. Repository: mesos Descriptio

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/#review97455 --- Patch looks great! Reviews applied: [36908, 36913] All tests passe

Re: Review Request 37500: Update the FrameworkInfo.user on scheduler failover

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37500/#review97456 --- src/slave/slave.cpp (line 2133)

Re: Review Request 37669: Ignore overflow docker components in version parsing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37669/#review97453 --- src/docker/docker.cpp (line 199)

Re: Review Request 37669: Ignore overflow docker components in version parsing.

> On Sept. 2, 2015, 2:54 p.m., Bernd Mathiske wrote: > > src/docker/docker.cpp, line 201 > > > > > > The "[0]" access looks dicy, but after reading the implementation of > > strings::split(), I am fairly convinced t

Re: Review Request 37669: Ignore overflow docker components in version parsing.

> On Sept. 2, 2015, 2:54 p.m., Bernd Mathiske wrote: > > src/docker/docker.cpp, line 201 > > > > > > The "[0]" access looks dicy, but after reading the implementation of > > strings::split(), I am fairly convinced t

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/#review97464 --- Using a ```RefusedFilter``` for this is not the right solution. The

Re: Review Request 38046: Replace hard-coded reap interval with a constant

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38046/#review97465 --- I've started to review this patch and then I realized that it was ve

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 Sept. 2, 2015, 4:16 p.m.) Review request for mesos, Alexander Rukletso

Re: Review Request 37873: Add quiesce logic in allocator

> On 九月 2, 2015, 4:07 p.m., James Peach wrote: > > Using a ```RefusedFilter``` for this is not the right solution. The point > > of the suppress call is to have a cheap way to disable offers at a > > framework granularity. Adding a ```RefusedFilter``` for the total is not > > cheap. Filters ar

Re: Review Request 38056: Fix webui task informations not update bug.

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

Review Request 38061: Corrected include order in master.hpp

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38061/ --- Review request for mesos and Joris Van Remoortere. Repository: mesos Descript

Re: Review Request 38046: Replace hard-coded reap interval with a constant

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38046/ --- (Updated 九月 2, 2015, 4:37 p.m.) Review request for mesos and Ben Mahler. Bugs

Re: Review Request 38061: Corrected include order in master.hpp

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38061/#review97475 --- src/master/master.hpp

Re: Review Request 37921: Add Copy backend for provisioners.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/ --- (Updated Sept. 2, 2015, 4:54 p.m.) Review request for mesos, Jie Yu and Jiang Y

Re: Review Request 38061: Corrected include order in master.hpp

> On Sept. 2, 2015, 4:45 p.m., Guangya Liu wrote: > > src/master/master.hpp, line 34 > > > > > > Alex, can you please show more detail for why we need to move #include > > ? Does the correct order is sub grouping >

Re: Review Request 37873: Add quiesce logic in allocator

> On Sept. 2, 2015, 4:07 p.m., James Peach wrote: > > Using a ```RefusedFilter``` for this is not the right solution. The point > > of the suppress call is to have a cheap way to disable offers at a > > framework granularity. Adding a ```RefusedFilter``` for the total is not > > cheap. Filters

Re: Review Request 37773: Docker: Adding registry client.

- Jojy --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37773/#review97331 --- On Sept. 1, 2015, 9:36 p.m., Jojy Varghese wrote: > > ---

Re: Review Request 37826: Introduced conversion of JSON arrays to repeated protobufs.

> On Sept. 1, 2015, 9:46 a.m., Joseph Wu wrote: > > 3rdparty/libprocess/3rdparty/stout/include/stout/protobuf.hpp, lines 553-555 > > > > > > Can you explain why this is a struct rather than a function? > > Alexander

Re: Review Request 38061: Corrected include order in master.hpp

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38061/#review97493 --- Ship it! Ship It! - Joseph Wu On Sept. 2, 2015, 9:34 a.m., Alexa

Re: Review Request 37773: Docker: Adding registry client.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37773/ --- (Updated Sept. 2, 2015, 5:06 p.m.) Review request for mesos, Lily Chen, Joris V

Re: Review Request 37427: Docker registry: adding TokenManager.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37427/ --- (Updated Sept. 2, 2015, 5:07 p.m.) Review request for mesos, Lily Chen, Joris V

Re: Review Request 38046: Replace hard-coded reap interval with a constant

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

Re: Review Request 34142: AppC provisioner.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34142/#review97499 --- This is no longer valid right? Can we discard all ian's patches? -

Re: Review Request 37993: Add explanatory comments for Allocator interface

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37993/#review97478 --- First of all, thanks for adding comments! include/mesos/master/all

Re: Review Request 37921: Add Copy backend for provisioners.

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

Re: Review Request 37462: Add support for version detection and parsing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37462/ --- (Updated Sept. 2, 2015, 6:52 p.m.) Review request for mesos and Ben Mahler. B

Re: Review Request 37462: Add support for version detection and parsing.

> On Sept. 1, 2015, 11:43 p.m., Ben Mahler wrote: > > src/linux/perf.hpp, line 56 > > > > > > Whoops, doesn't this comment belong on the parse function? Actually, it applies to both of them. I'll make that clear.

Re: Review Request 37170: Maintenance Primitives: Added `Machine` to Slave struct in Master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37170/ --- (Updated Sept. 2, 2015, 7:31 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37176: Maintenance Primitives: Added a new allocation overload to sorter.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37176/ --- (Updated Sept. 2, 2015, 7:31 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37773: Docker: Adding registry client.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37773/#review97518 --- Patch looks great! Reviews applied: [37871, 37427, 37773] All test

Re: Review Request 37173: Maintenance Primitives: Added unavailability to Allocator's Slave struct.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37173/ --- (Updated Sept. 2, 2015, 7:31 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37180: Maintenance Primitives: Implemented Master::inverseOffer.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37180/ --- (Updated Sept. 2, 2015, 7:32 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37282: Maintenance Primitives: Added InverseOffer to V1 API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37282/ --- (Updated Sept. 2, 2015, 7:32 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37283: Maintenance Primitives: Refactored Master maintenance test to use V1 API.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37283/ --- (Updated Sept. 2, 2015, 7:32 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37175: Maintenance Primitives: Added updateUnavailability to master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37175/ --- (Updated Sept. 2, 2015, 7:31 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37178: Maintenance Primitives: Added InverseOffers to Scheduler Event Offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37178/ --- (Updated Sept. 2, 2015, 7:32 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37177: Maintenance Primitives: Added inverse offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37177/ --- (Updated Sept. 2, 2015, 7:32 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37172: Maintenance Primitives: Set offer `unavailability` if slave is scheduled for maintenance.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37172/ --- (Updated Sept. 2, 2015, 7:31 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37621: Maintenance Primitives: Gracefully handle inverse offers in pre-V1 scheduler.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37621/ --- (Updated Sept. 2, 2015, 7:33 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37234: Maintenance Primitives: Added URL field to InverseOffer proto.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37234/ --- (Updated Sept. 2, 2015, 7:32 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37622: Maintenance Primitives: Shutdown & remove slave when maintenance is started.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37622/ --- (Updated Sept. 2, 2015, 7:33 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37280: Maintenance Primitives: Added updateInverseOffer to Allocator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37280/ --- (Updated Sept. 2, 2015, 7:32 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37284: Maintenance Primitives: Added support for Accept / Decline of InverseOffers in master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37284/ --- (Updated Sept. 2, 2015, 7:33 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37623: Maintenance Primitives: Prevent Slaves from registering if the machine is under maintenance.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37623/ --- (Updated Sept. 2, 2015, 7:33 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 37427: Docker registry: adding TokenManager.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37427/#review97505 --- Nothing major, Just some fly-by style comments. Looking at the imple

Re: Review Request 37466: Update perf tests to including testing the supported perf output formats.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37466/ --- (Updated Sept. 2, 2015, 8:06 p.m.) Review request for mesos and Ben Mahler. C

Re: Review Request 38061: Corrected include order in master.hpp

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

Re: Review Request 37623: Maintenance Primitives: Prevent Slaves from registering if the machine is under maintenance.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37623/#review97528 --- Bad patch! Reviews applied: [37655, 36321, 36571, 37314, 37900, 373

Re: Review Request 37466: Update perf tests to including testing the supported perf output formats.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37466/#review97530 --- Bad patch! Reviews applied: [37416, 37442, 37462, 37466] Failed co

Re: Review Request 37427: Docker registry: adding TokenManager.

> On Sept. 2, 2015, 8:01 p.m., Anand Mazumdar wrote: > > Nothing major, Just some fly-by style comments. Looking at the > > implementation in greater detail now. > > > > I wonder if we should split this review into smaller chunks. It currently > > stands at ~900 lines. It's very hard to review

Review Request 38069: Added a tuple overload for process::collect.

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

Re: Review Request 37170: Maintenance Primitives: Added `Machine` to Slave struct in Master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37170/#review97520 --- Ship it! LGTM! src/master/master.cpp (lines 1367 - 1369)

Re: Review Request 37466: Update perf tests to including testing the supported perf output formats.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37466/ --- (Updated Sept. 2, 2015, 9:13 p.m.) Review request for mesos and Ben Mahler. C

Re: Review Request 38069: Added a tuple overload for process::collect.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38069/#review97534 --- Ship it! Ship It! - Vinod Kone On Sept. 2, 2015, 9:09 p.m., Ben

Re: Review Request 37427: Docker registry: adding TokenManager.

> On Sept. 2, 2015, 8:01 p.m., Anand Mazumdar wrote: > > src/slave/containerizer/provisioners/docker/token_manager.hpp, line 19 > > > > > > Can we change this to be __MESOS_DOCKER_TOKEN_MANAGER_HPP__ > > > >

Re: Review Request 37921: Add Copy backend for provisioners.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37921/#review97509 --- Ship it! src/slave/containerizer/provisioners/backends/copy.hpp (l

Re: Review Request 37466: Update perf tests to including testing the supported perf output formats.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37466/#review97538 --- Patch looks great! Reviews applied: [37416, 37442, 37462, 37466] A

Re: Review Request 37427: Docker registry: adding TokenManager.

> On Sept. 2, 2015, 8:01 p.m., Anand Mazumdar wrote: > > src/slave/containerizer/provisioners/docker/token_manager.hpp, line 19 > > > > > > Can we change this to be __MESOS_DOCKER_TOKEN_MANAGER_HPP__ > > > >

Re: Review Request 37172: Maintenance Primitives: Set offer `unavailability` if slave is scheduled for maintenance.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37172/#review97524 --- Ship it! LGTM. (And I think BenH's issue is no longer present eit

Re: Review Request 37462: Add support for version detection and parsing.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37462/#review97516 --- Ship it! I've made adjustments based on the feedback below before c

Re: Review Request 37175: Maintenance Primitives: Added updateUnavailability to master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37175/#review97527 --- Ship it! include/mesos/master/allocator.hpp (lines 138 - 141)

Re: Review Request 37177: Maintenance Primitives: Added inverse offers.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37177/#review97529 --- Ship it! LGTM! src/master/allocator/mesos/hierarchical.hpp (line

Re: Review Request 37280: Maintenance Primitives: Added updateInverseOffer to Allocator.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37280/#review97532 --- Ship it! include/mesos/master/allocator.hpp (lines 152 - 155)

Re: Review Request 37540: Add PerfEvent API

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37540/ --- (Updated Sept. 2, 2015, 10:13 p.m.) Review request for mesos, Chi Zhang, Ian Do

Re: Review Request 37541: Add TraceEvent API

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37541/ --- (Updated Sept. 2, 2015, 10:14 p.m.) Review request for mesos, Chi Zhang, Ian Do

Re: Review Request 37427: Docker registry: adding TokenManager.

> On Sept. 2, 2015, 8:01 p.m., Anand Mazumdar wrote: > > src/slave/containerizer/provisioners/docker/token_manager.hpp, line 19 > > > > > > Can we change this to be __MESOS_DOCKER_TOKEN_MANAGER_HPP__ > > > >

Re: Review Request 37541: Add trace event API

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37541/ --- (Updated Sept. 2, 2015, 10:17 p.m.) Review request for mesos, Chi Zhang, Ian Do

Re: Review Request 37540: Add perf event API

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37540/ --- (Updated Sept. 2, 2015, 10:16 p.m.) Review request for mesos, Chi Zhang, Ian Do

Review Request 38073: Avoid parsing perf events twice

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

Re: Review Request 38073: Avoid parsing perf events twice

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38073/#review97546 --- Ship it! Ship It! - Ben Mahler On Sept. 2, 2015, 10:18 p.m., Con

Review Request 38074: Calculate schedule latency with trace events

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

Re: Review Request 38074: Calculate schedule latency with trace events

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

Re: Review Request 37881: Implemented AppcProvisioner.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37881/ --- (Updated Sept. 2, 2015, 3:47 p.m.) Review request for mesos, Chi Zhang, Jie Yu,

Re: Review Request 37466: Update perf tests to including testing the supported perf output formats.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37466/#review97551 --- Thinking about this further, the test parameteratization here seems

Re: Review Request 37427: Docker registry: adding TokenManager.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37427/ --- (Updated Sept. 2, 2015, 10:50 p.m.) Review request for mesos, Lily Chen, Joris

Re: Review Request 37773: Docker: Adding registry client.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37773/ --- (Updated Sept. 2, 2015, 10:52 p.m.) Review request for mesos, Lily Chen, Joris

Re: Review Request 37284: Maintenance Primitives: Added support for Accept / Decline of InverseOffers in master.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37284/#review97553 --- Ship it! src/master/master.cpp (line 3243)

Re: Review Request 38074: Calculate schedule latency with trace events

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38074/ --- (Updated Sept. 2, 2015, 11:18 p.m.) Review request for mesos, Chi Zhang, Ian Do

Re: Review Request 38074: Calculate schedule latency with trace events

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38074/ --- (Updated Sept. 2, 2015, 11:19 p.m.) Review request for mesos, Chi Zhang, Ian Do

Re: Review Request 37622: Maintenance Primitives: Shutdown & remove slave when maintenance is started.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37622/#review97555 --- Ship it! src/master/http.cpp (line 1567)

  1   2   >