mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Meng Zhu <m...@mesosphere.io>
Subject Review Request 69068: Added `hasGpu` flag in the allocator `Slave` struct.
Date Wed, 17 Oct 2018 23:41:40 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69068/
-----------------------------------------------------------

Review request for mesos, Benjamin Mahler and Gastón Kleiman.


Repository: mesos


Description
-------

Due to the `filterGpuResources` mechanism, before making any
allocations, the allocator needs to check if an agent has
any GPU resources. This is currently done by calling the
`Resources::gpus()` function which traverse the resource
vector and copy the scalar. Since this is frequently called
and agent total resources rarely change, caching this boolean
will yield quite some performance improvement.


Diffs
-----

  src/master/allocator/mesos/hierarchical.hpp e09a615979014909e7b88d44e6617fd1476415c5 
  src/master/allocator/mesos/hierarchical.cpp 6f389ed46683f7c642e87b673dc15307cc2b5cd6 


Diff: https://reviews.apache.org/r/69068/diff/1/


Testing
-------

make check


Thanks,

Meng Zhu


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