Re: Review Request 37074: Added process id prefix for benchmarks and process tests.

2015-08-06 Thread Alexander Rojas

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

Ship it!


Ship It!

- Alexander Rojas


On Aug. 5, 2015, 4:52 p.m., Till Toenshoff wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37074/
> ---
> 
> (Updated Aug. 5, 2015, 4:52 p.m.)
> 
> 
> Review request for mesos.
> 
> 
> Bugs: MESOS-1457
> https://issues.apache.org/jira/browse/MESOS-1457
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> This patch was originally proposed by Palak Choudhary via 
> https://reviews.apache.org/r/31076/. This version is just rebased onto the 
> current master.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/src/tests/benchmarks.cpp 
> 97c81b8e61a75771e5bf7d46505cec4e0c0f404a 
>   3rdparty/libprocess/src/tests/process_tests.cpp 
> 95e3257b030128e9d03dde9aa048602c68c6a446 
> 
> Diff: https://reviews.apache.org/r/37074/diff/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Till Toenshoff
> 
>



Re: Review Request 37074: Added process id prefix for benchmarks and process tests.

2015-08-05 Thread Mesos ReviewBot

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


Patch looks great!

Reviews applied: [37074]

All tests passed.

- Mesos ReviewBot


On Aug. 5, 2015, 2:52 p.m., Till Toenshoff wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37074/
> ---
> 
> (Updated Aug. 5, 2015, 2:52 p.m.)
> 
> 
> Review request for mesos.
> 
> 
> Bugs: MESOS-1457
> https://issues.apache.org/jira/browse/MESOS-1457
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> This patch was originally proposed by Palak Choudhary via 
> https://reviews.apache.org/r/31076/. This version is just rebased onto the 
> current master.
> 
> 
> Diffs
> -
> 
>   3rdparty/libprocess/src/tests/benchmarks.cpp 
> 97c81b8e61a75771e5bf7d46505cec4e0c0f404a 
>   3rdparty/libprocess/src/tests/process_tests.cpp 
> 95e3257b030128e9d03dde9aa048602c68c6a446 
> 
> Diff: https://reviews.apache.org/r/37074/diff/
> 
> 
> Testing
> ---
> 
> make check
> 
> 
> Thanks,
> 
> Till Toenshoff
> 
>



Review Request 37074: Added process id prefix for benchmarks and process tests.

2015-08-05 Thread Till Toenshoff

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

Review request for mesos.


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


Repository: mesos


Description
---

This patch was originally proposed by Palak Choudhary via 
https://reviews.apache.org/r/31076/. This version is just rebased onto the 
current master.


Diffs
-

  3rdparty/libprocess/src/tests/benchmarks.cpp 
97c81b8e61a75771e5bf7d46505cec4e0c0f404a 
  3rdparty/libprocess/src/tests/process_tests.cpp 
95e3257b030128e9d03dde9aa048602c68c6a446 

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


Testing
---

make check


Thanks,

Till Toenshoff