mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexander Rojas" <alexan...@mesosphere.io>
Subject Re: Review Request 37996: Added property manager
Date Tue, 01 Sep 2015 12:26:18 GMT

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

(Updated Sept. 1, 2015, 2:26 p.m.)


Review request for mesos and Benjamin Hindman.


Changes
-------

Removes use of `auto` and for range loops.


Repository: mesos


Description
-------

Introduces the `PathInheritedProperties` class which allows to create a tree where nodes can
be tag with some property. This property is then inherited by children nodes.

Two behaviors are implemented, overriding, i.e. Adding a property to the child node of another
node with a property already will result in the ancestor property being lost. The second behavior,
accumulating, takes a function and accumulates
properties of all ancestors.


Diffs (updated)
-----

  3rdparty/libprocess/3rdparty/stout/Makefile.am f95ed03004d4e5382874d75969bc9285a0f44918

  3rdparty/libprocess/3rdparty/stout/include/Makefile.am 8853f92fcfcff81d0a3197bade02110685fa0325

  3rdparty/libprocess/3rdparty/stout/include/stout/properties.hpp PRE-CREATION 
  3rdparty/libprocess/3rdparty/stout/tests/CMakeLists.txt b1a519007e5bed196541f294c4676277f9708714

  3rdparty/libprocess/3rdparty/stout/tests/properties_tests.cpp PRE-CREATION 

Diff: https://reviews.apache.org/r/37996/diff/


Testing
-------

make check


Thanks,

Alexander Rojas


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