----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43367/#review118576 -----------------------------------------------------------
src/tests/fetcher_cache_tests.cpp (line 64) <https://reviews.apache.org/r/43367/#comment179848> Do you need the DEFAULT_ALLOCATION_INTERVAL namespace? `using mesos::internal::master::DEFAULT_ALLOCATION_INTERVAL;` - Guangya Liu On 二月 10, 2016, 4:01 a.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43367/ > ----------------------------------------------------------- > > (Updated 二月 10, 2016, 4:01 a.m.) > > > Review request for mesos and Bernd Mathiske. > > > Bugs: MESOS-4618 > https://issues.apache.org/jira/browse/MESOS-4618 > > > Repository: mesos > > > Description > ------- > > Speed up FetcherCache test cases by advance allocation_interval. > > > Diffs > ----- > > src/tests/fetcher_cache_tests.cpp 1cf45660691860793ac600363f7934e13a2e7ddf > > Diff: https://reviews.apache.org/r/43367/diff/ > > > Testing > ------- > > Before this patch: > ``` > [ OK ] FetcherCacheTest.LocalUncached (1065 ms) > [ OK ] FetcherCacheTest.LocalCached (1728 ms) > [ OK ] FetcherCacheTest.CachedFallback (1105 ms) > [ OK ] FetcherCacheTest.LocalUncachedExtract (906 ms) > [ OK ] FetcherCacheTest.LocalCachedExtract (1715 ms) > [ OK ] FetcherCacheTest.SimpleEviction (3626 ms) > [ OK ] FetcherCacheTest.FallbackFromEviction (2649 ms) > [ OK ] FetcherCacheTest.RemoveLRUCacheEntries (3639 ms) > [ OK ] FetcherCacheHttpTest.HttpCachedSerialized (2608 ms) > [ OK ] FetcherCacheHttpTest.HttpCachedConcurrent (1529 ms) > [ OK ] FetcherCacheHttpTest.HttpMixed (1349 ms) > ``` > > After this patch: > ``` > [ OK ] FetcherCacheTest.LocalUncached (1132 ms) > [ OK ] FetcherCacheTest.LocalCached (1693 ms) > [ OK ] FetcherCacheTest.CachedFallback (1028 ms) > [ OK ] FetcherCacheTest.LocalUncachedExtract (998 ms) > [ OK ] FetcherCacheTest.LocalCachedExtract (1341 ms) > [ OK ] FetcherCacheTest.SimpleEviction (2056 ms) > [ OK ] FetcherCacheTest.FallbackFromEviction (2146 ms) > [ OK ] FetcherCacheTest.RemoveLRUCacheEntries (2576 ms) > [ OK ] FetcherCacheHttpTest.HttpCachedSerialized (1588 ms) > [ OK ] FetcherCacheHttpTest.HttpCachedConcurrent (1453 ms) > [ OK ] FetcherCacheHttpTest.HttpMixed (1143 ms) > ``` > > > Thanks, > > haosdent huang > >