mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Re: Review Request 64163: Made sure only non-terminal operations use resources.
Date Thu, 30 Nov 2017 16:00:32 GMT

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

(Updated Nov. 30, 2017, 5 p.m.)

Review request for mesos, Jie Yu and Jan Schlicht.



Repository: mesos


The function 'addOfferOperation' is a high-level function which can be
used to some extent manage the offer operation life cycle in the
master. In addition to managing the lifetime of offer operations, it
also maintains the state of used resources.

We already made sure to never count speculated operations as using
resources in this function. In addition, the function
'updateOfferOperation' is already capable of updating the used
resources when an offer operation becomes terminal.  To complete this
division of responsibilities we in this patch extend the conditional
treatment in 'removeOfferOperation' to also take into account whether
the operation is terminal when deciding on whether to mutate the used

Diffs (updated)

  src/master/master.cpp fadc78b2ca5d46b8cc12a794b428753aa79ac095 




`make check`


Benjamin Bannier

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