Re: Review Request 57110: Updated the master to handle frameworks that changes its roles.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57110/ --- (Updated March 4, 2017, 7:55 p.m.) Review request for mesos and Benjamin

Re: Review Request 57111: Updated the allocator to handle frameworks that change its roles.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57111/ --- (Updated March 4, 2017, 7:36 p.m.) Review request for mesos and Benjamin

Re: Review Request 57110: Updated the master to handle frameworks that changes its roles.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57110/ --- (Updated March 4, 2017, 7:31 p.m.) Review request for mesos and Benjamin

Re: Review Request 57319: Fixed a bug around temporaries in `src/master/http.cpp`.

2017-03-04 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57319/#review167940 --- Ship it! Ship It! - Neil Conway On March 5, 2017, 3:23

Review Request 57319: Fixed a bug around temporaries in `src/master/http.cpp`.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57319/ --- Review request for mesos, Benjamin Mahler and Neil Conway. Repository: mesos

Re: Review Request 57111: Updated the allocator to handle frameworks that change its roles.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57111/ --- (Updated March 4, 2017, 1:32 p.m.) Review request for mesos and Benjamin

Re: Review Request 57110: Updated the master to handle frameworks that changes its roles.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57110/ --- (Updated March 4, 2017, 1:09 p.m.) Review request for mesos and Benjamin

Re: Review Request 57110: Updated the master to handle frameworks that changes its roles.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57110/ --- (Updated March 4, 2017, 11:53 a.m.) Review request for mesos and Benjamin

Re: Review Request 57111: Updated the allocator to handle frameworks that change its roles.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57111/ --- (Updated March 4, 2017, 11:47 a.m.) Review request for mesos and Benjamin

Re: Review Request 57111: Updated the allocator to handle frameworks that change its roles.

2017-03-04 Thread Michael Park
> On March 3, 2017, 5:58 p.m., Benjamin Mahler wrote: > > src/master/allocator/mesos/hierarchical.cpp > > Line 458 (original), 463 (patched) > > > > > > Can you commit this separately? Committed separately here:

Re: Review Request 57282: Sent `UpdateFrameworkMessage` even to disconnected frameworks.

2017-03-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57282/ --- (Updated March 4, 2017, 9:37 a.m.) Review request for mesos and Benjamin