Re: Review Request 42252: Enabled master totalResources include allocation slack.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42252/#review115401 --- Patch looks great! Reviews applied: [40375, 41334, 41333, 40529,

Re: Review Request 42252: Enabled master totalResources include allocation slack.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42252/#review115292 --- Considering that https://reviews.apache.org/r/41847/ has the same

Re: Review Request 42252: Enabled master totalResources include allocation slack.

> On 一月 20, 2016, 12:16 a.m., Joseph Wu wrote: > > Considering that https://reviews.apache.org/r/41847/ has the same change, > > it may be appropriate to add a `Resources::nonUsageSlack()` helper. Added a helper here https://reviews.apache.org/r/42547/ - Guangya

Re: Review Request 42252: Enabled master totalResources include allocation slack.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42252/#review115341 --- Patch looks great! Reviews applied: [40375, 41334, 41333, 40529,

Re: Review Request 42252: Enabled master totalResources include allocation slack.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42252/#review115339 --- src/master/master.cpp (line 4659)

Re: Review Request 42252: Enabled master totalResources include allocation slack.

> On 一月 20, 2016, 3:43 a.m., Klaus Ma wrote: > > src/master/master.cpp, line 4659 > > > > > > It checked Optimistic Offer Phase 1 is enabled in > > https://reviews.apache.org/r/41847/. But I think it's not

Re: Review Request 42252: Enabled master totalResources include allocation slack.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42252/ --- (Updated 一月 20, 2016, 7:29 a.m.) Review request for mesos, Ben Mahler, Artem