mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Timothy Chen" <tnac...@apache.org>
Subject Re: Review Request 24270: New MultiChildProcess test added to reap_tests.cpp
Date Tue, 05 Aug 2014 21:43:28 GMT


> On Aug. 5, 2014, 12:30 a.m., Ian Downes wrote:
> > I'm not 100% clear from the description what this is doing - does it test correctness
in anyway or is it really to be used under profiling to test the performance of the reaper?
If it's the latter, then perhaps this should be standalone benchmark that steps through #
of child pids?
> 
> Craig Hansen-Sturm wrote:
>     This is meant for benchmarking; however, it is has dual purpose, and it wasn't clear
how to integrate this type of benchmark.   Ideally, we'd want to continuously vary the pid
count and, for this benchmark, the reap wait time.   The wait time isn't programmatically
exposed, but the question I have for you, is do we have a standard mechanism for exposing
"endpoints" such as this ? Create a new internal interface ?
>     
>     Finally, ideally, this test would make use of an external profiling api - so that
we can start/stop the sampling periods at test entry.
>     
>     I'm hoping GTEST has a lot of these hooks, but want to hear your ideas.

Just came to my mind that we already have a make bench call that runs BENCHMARK tests I believe.
Can this test then fold under benchmark with a BENCHMARK tag as well?


- Timothy


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


On Aug. 4, 2014, 11:23 p.m., Craig Hansen-Sturm wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/24270/
> -----------------------------------------------------------
> 
> (Updated Aug. 4, 2014, 11:23 p.m.)
> 
> 
> Review request for mesos and Ian Downes.
> 
> 
> Bugs: MESOS-1669
>     https://issues.apache.org/jira/browse/MESOS-1669
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> New test added which serves as a performance test for reaping. This test forks N children
(default value is 4), kills these children, and then uses process::collect to collect all
status results. This is a simple generalization of the existing ChildProcess test, except
that it is realtime (as required for profiling), e.g., no clock manipulation is performed.
 This test is also useful for stressing the collect.hpp interfaces.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/src/tests/reap_tests.cpp a18d54c 
> 
> Diff: https://reviews.apache.org/r/24270/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Craig Hansen-Sturm
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message