mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ilya Pronin <ipro...@twopensource.com>
Subject Review Request 56086: Simplified AppC provisioner cache keys comparison.
Date Mon, 30 Jan 2017 17:27:40 GMT

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

Review request for mesos, Benjamin Mahler and Jie Yu.


Repository: mesos


Description
-------

Manual elements searching is not required since `std::map` is ordered. Replaced it with standard
`operator==` that checks maps sizes and does linear element by element comparison.


Diffs
-----

  src/slave/containerizer/mesos/provisioner/appc/cache.cpp fed5ae97419d5eb9611a6fd1ade1bbed7ed440a2


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


Testing
-------

`make check`


Thanks,

Ilya Pronin


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