You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Joerg Schad <jo...@mesosphere.io> on 2016/03/28 14:05:30 UTC

Re: Review Request 45233: Refactored perf test without setup function [4/7].

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

(Updated March 28, 2016, 12:05 p.m.)


Review request for mesos and Joris Van Remoortere.


Changes
-------

Addressed review.


Summary (updated)
-----------------

Refactored perf test without setup function [4/7].


Repository: mesos


Description (updated)
-------

With the newly introduced watchdog option there is no need
for the child setup function in the perf code anymore.


Diffs (updated)
-----

  src/linux/perf.cpp 1c113a2b3f57877e132bbd65e01fb2f045132128 

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


Testing
-------

Tested entire chain (see https://reviews.apache.org/r/45236/).


Thanks,

Joerg Schad


Re: Review Request 45233: Refactored perf test without setup function [4/7].

Posted by Joerg Schad <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45233/
-----------------------------------------------------------

(Updated March 28, 2016, 4:52 p.m.)


Review request for mesos and Joris Van Remoortere.


Bugs: MESOS-5049
    https://issues.apache.org/jira/browse/MESOS-5049


Repository: mesos


Description
-------

With the newly introduced watchdog option there is no need
for the child setup function in the perf code anymore.


Diffs
-----

  src/linux/perf.cpp 1c113a2b3f57877e132bbd65e01fb2f045132128 

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


Testing
-------

Tested entire chain (see https://reviews.apache.org/r/45236/).


Thanks,

Joerg Schad