Re: Review Request 50172: Made `ports` resource configurable in sorter benchmark test.

2016-07-22 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50172/#review143251 --- Ship it! The commit summary and description seem to not

Re: Review Request 50172: Made `ports` resource configurable in sorter benchmark test.

2016-07-19 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50172/#review142773 --- Patch looks great! Reviews applied: [50172] Passed command:

Re: Review Request 50172: Made `ports` resource configurable in sorter benchmark test.

2016-07-19 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50172/ --- (Updated 七月 19, 2016, 3:59 p.m.) Review request for mesos and Benjamin Mahler.

Re: Review Request 50172: Made `ports` resource configurable in sorter benchmark test.

2016-07-19 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50172/#review142702 --- Patch looks great! Reviews applied: [50172] Passed command:

Re: Review Request 50172: Made `ports` resource configurable in sorter benchmark test.

2016-07-18 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50172/ --- (Updated 七月 19, 2016, 3:35 a.m.) Review request for mesos and Benjamin Mahler.

Re: Review Request 50172: Made `ports` resource configurable in sorter benchmark test.

2016-07-18 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50172/#review142684 --- src/tests/sorter_tests.cpp (lines 504 - 506)