Re: Review Request 36605: Cleaned up 'perf' related tests to work on Ubuntu 14.04.

2015-07-19 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36605/#review92213 --- Ship it! - Joerg Schad On July 19, 2015, 6:04 p.m., Benjamin

Re: Review Request 36605: Cleaned up 'perf' related tests to work on Ubuntu 14.04.

2015-07-19 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36605/#review92210 --- src/tests/cgroups_tests.cpp (line 991)

Review Request 36605: Cleaned up 'perf' related tests to work on Ubuntu 14.04.

2015-07-19 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36605/ --- Review request for mesos, Ian Downes and Timothy Chen. Bugs: MESOS-3079

Re: Review Request 36605: Cleaned up 'perf' related tests to work on Ubuntu 14.04.

2015-07-19 Thread Timothy Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36605/#review92215 --- Ship it! Ship It! - Timothy Chen On July 19, 2015, 6:04 p.m.,

Re: Review Request 36605: Cleaned up 'perf' related tests to work on Ubuntu 14.04.

2015-07-19 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36605/ --- (Updated July 19, 2015, 6:04 p.m.) Review request for mesos, Ian Downes and

Re: Review Request 36605: Cleaned up 'perf' related tests to work on Ubuntu 14.04.

2015-07-19 Thread Benjamin Hindman
On July 19, 2015, 5:55 p.m., Timothy Chen wrote: src/tests/cgroups_tests.cpp, line 992 https://reviews.apache.org/r/36605/diff/1/?file=1016074#file1016074line992 I wonder what is the follow up action around this TODO? Seems like it stated why we commented the code but not