Re: Review Request 32664: Add port mapping isolator statistics tests

2015-05-27 Thread Paul Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32664/ --- (Updated May 27, 2015, 9:04 p.m.) Review request for mesos, Chi Zhang, Ian

Re: Review Request 32664: Add port mapping isolator statistics tests

2015-05-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32664/#review85512 --- Bad patch! Reviews applied: [32664] Failed command:

Re: Review Request 32664: Add port mapping isolator statistics tests

2015-05-22 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32664/#review85032 --- Patch looks great! Reviews applied: [34321, 34426, 34428, 34431,

Re: Review Request 32664: Add port mapping isolator statistics tests

2015-05-22 Thread Paul Brett
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32664/ --- (Updated May 22, 2015, 11:45 p.m.) Review request for mesos, Chi Zhang, Ian

Re: Review Request 32664: Add port mapping isolator statistics tests

2015-05-21 Thread Paul Brett
On April 8, 2015, 7:46 p.m., Ian Downes wrote: src/tests/port_mapping_tests.cpp, line 1788 https://reviews.apache.org/r/32664/diff/1/?file=911866#file911866line1788 Did you consider using iperf3 which makes all features available through a library? I looked at iperf3 but it