> On Feb. 16, 2016, 3:02 a.m., Alexander Rukletsov wrote:
> > Could you please explain in description why disabling checkpointing speeds 
> > up the test and why it is ok to disable it for these tests?
> 
> haosdent huang wrote:
>     Sure!

Not writing to disk costs less time.

If a test does not attempt recovery, then whatever would be checkpointed never 
gets used.

Only one test attempts/tests recovery so far. In this test we do apply 
checkpointing. So it does get tested, but we don't need to test the writing 
part of it in every other test.


- Bernd


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


On Feb. 9, 2016, 4:48 p.m., haosdent huang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42696/
> -----------------------------------------------------------
> 
> (Updated Feb. 9, 2016, 4:48 p.m.)
> 
> 
> Review request for mesos and Bernd Mathiske.
> 
> 
> Bugs: MESOS-4486
>     https://issues.apache.org/jira/browse/MESOS-4486
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Speed up FetcherCacheTest.Local* test cases.
> 
> 
> Diffs
> -----
> 
>   src/tests/fetcher_cache_tests.cpp 1cf45660691860793ac600363f7934e13a2e7ddf 
> 
> Diff: https://reviews.apache.org/r/42696/diff/
> 
> 
> Testing
> -------
> 
> Before apply the patch:
> ```
> $ sudo ./bin/mesos-tests.sh --gtest_filter="FetcherCacheTest.Local*"
> [       OK ] FetcherCacheTest.LocalUncached (2580 ms)
> [       OK ] FetcherCacheTest.LocalCached (2524 ms)
> [       OK ] FetcherCacheTest.LocalUncachedExtract (2514 ms)
> [       OK ] FetcherCacheTest.LocalCachedExtract (2551 ms)
> ```
> 
> 
> After apply the patch:
> ```
> $ sudo ./bin/mesos-tests.sh --gtest_filter="FetcherCacheTest.Local*"
> [       OK ] FetcherCacheTest.LocalUncached (873 ms)
> [       OK ] FetcherCacheTest.LocalCached (1609 ms)
> [       OK ] FetcherCacheTest.LocalUncachedExtract (926 ms)
> [       OK ] FetcherCacheTest.LocalCachedExtract (1509 ms)
> ```
> 
> Also test the recovery test manually:
> 
> ```
> sudo ./bin/mesos-tests.sh 
> --gtest_filter="FetcherCacheHttpRecoveryTest.HttpCachedRecovery"
> ```
> 
> 
> Thanks,
> 
> haosdent huang
> 
>

Reply via email to