mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael Park <>
Subject Review Request 60282: Introduced `upgradeResources` which also validates the resources.
Date Wed, 21 Jun 2017 19:07:20 GMT

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

Review request for mesos and Neil Conway.

Repository: mesos


Validation code for operations, and tasks validated the resources as
the first step, and assumed valid resources from then on. This means
that it started to use functions such as `isDynamicallyReserved`.

However, since `isDynamicallyReserved` requires the resources to be
in the "post-reservation-refinement" format, we must convert the
resources before using those functions. For now, we introduce
a `upgradeResources` abstraction which is called to validate and
convert the resources before invoking operation / task validation.


  src/common/resources_utils.hpp 4b033687fa7224ffa36523cb3ca366f4384fd4f8 
  src/common/resources_utils.cpp 751194c5ff5c794c77be3bcf16892af39de6d4d9 




Michael Park

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