Re: Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-30 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36828/#review93654 --- Ship it! I'll fix up and commit, thanks! 3rdparty/libprocess/3rdp

Re: Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-27 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36828/ --- (Updated July 27, 2015, 8:45 p.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-27 Thread Joris Van Remoortere
> On July 27, 2015, 7:21 p.m., Benjamin Hindman wrote: > > 3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp, line 142 > > > > > > Same comments here as previous reviews: use Latch instead of condition > > var

Re: Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-27 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36828/#review93167 --- 3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp (line 107)

Re: Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-27 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36828/ --- (Updated July 27, 2015, 7:04 a.m.) Review request for mesos, Benjamin Hindman,

Re: Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-26 Thread Artem Harutyunyan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36828/#review93080 --- Ship it! 3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp (

Re: Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-26 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36828/#review93059 --- Patch looks great! Reviews applied: [36822, 36823, 36824, 36825, 36

Review Request 36828: Used std::thread instead of pthread for stout proc tests.

2015-07-26 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36828/ --- Review request for mesos, Benjamin Hindman, Artem Harutyunyan, and Michael Park.