mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrei Sekretenko <>
Subject Re: Review Request 72897: Made offer constraints filter and protobuf non-optional inside the code.
Date Thu, 24 Sep 2020 15:07:31 GMT

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

(Updated Sept. 24, 2020, 5:07 p.m.)

Review request for mesos and Benjamin Mahler.


Removed filter/constraints optionality instead of the previous approach.

Summary (updated)

Made offer constraints filter and protobuf non-optional inside the code.

Bugs: MESOS-10189

Repository: mesos

Description (updated)

Given that Mesos now provides a guarantee that specifying no offer
constraint in UPDATE_FRAMEWORK/SUBSCRIBE call is equivalent to
specifying default-constructed `OfferConstraints`, and that we are
intending to make the V0 scheduler driver always require offer
constraints as an argument to the `updateFramework()`, it no longer
makes sense to keep `OfferConstraints`/`OfferConstraintsFilter`
optional inside the Mesos code base.

This patch replaces a non-set `Option<OfferConstraints>` with
default-constructed `OfferConstraints`, and a non-set
`Option<OfferConstraintsFilter>` with a default-constructed filter.

Diffs (updated)

  include/mesos/allocator/allocator.hpp b0a5d6aaa08a05cb1ece318999a8760df932ba64 
  src/master/allocator/mesos/offer_constraints_filter.cpp 441ebc10219bf3bd623fac2bb08945ea9deb7ea3

  src/master/framework.cpp 980828e04cae96f091a6b30e553ff667fa58a917 
  src/master/http.cpp 3d8e7095ee58c2505042095b2c593c2317f3232f 
  src/master/master.hpp 95aca586f17a206b6ac7f0c48ea99c8525c336ad 
  src/master/master.cpp 576ae107a6a3c99b51516fdde4df9bff86c90a99 
  src/master/readonly_handler.cpp df499d1a5e9622222ee2fe0c47bc77fb1ef9d8bc 





Andrei Sekretenko

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