mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anindya Sinha <>
Subject Re: Review Request 55359: Consolidate update of allocations in `updateAllocation()`.
Date Tue, 07 Feb 2017 05:02:01 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 7, 2017, 5:02 a.m.)

Review request for mesos, Benjamin Mahler and Jiang Yan Xu.


Addressed comment and rebased.

Bugs: MESOS-6444

Repository: mesos


We handle shared resources for `LAUNCH` operations separately in the
`updateAllocation()` API to add additional copies of shared resources.
But the updates to the allocations in the allocator and sorters
can be consolidated together.

Diffs (updated)

  src/master/allocator/mesos/hierarchical.cpp 56d6791baa64189523df668749f4a7ab67d6b363 



Tests passed.


Anindya Sinha

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message