You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Alexander Rukletsov <ru...@gmail.com> on 2016/04/13 12:07:53 UTC

Re: Review Request 44336: Removed numeric suffixes where appropriate in allocator tests.

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

(Updated April 13, 2016, 10:07 a.m.)


Review request for mesos, Bernd Mathiske and Ben Mahler.


Changes
-------

Rebased.


Repository: mesos


Description
-------

We should not enumerate variables if the corresponding instance is the
only one of such type in the test.


Diffs (updated)
-----

  src/tests/hierarchical_allocator_tests.cpp 03064da4a5feda0d64db6175c7d5a8e3122bb67a 

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


Testing
-------

On Mac OS 10.10.4:

`make check`

`GTEST_FILTER="HierarchicalAllocatorTest.*" ./bin/mesos-tests.sh --gtest_repeat=100 --gtest_break_on_failure --gtest_shuffle`


Thanks,

Alexander Rukletsov