----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51794/#review148641 -----------------------------------------------------------
src/tests/containerizer/cgroups_tests.cpp (line 512) <https://reviews.apache.org/r/51794/#comment216130> Suggest to rename the test from `ROOT_CGROUPS_CFS_Big_Quota` to `ROOT_CGROUPS_CFS_Big_Quota_Decimal` src/tests/containerizer/cgroups_tests.cpp (lines 517 - 523) <https://reviews.apache.org/r/51794/#comment216134> I think we may not need these code anymore since `100.5` has already covered both `big` and `decimal`. - Qian Zhang On Sept. 12, 2016, 12:46 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51794/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2016, 12:46 p.m.) > > > Review request for mesos, Gilbert Song, Jie Yu, and Qian Zhang. > > > Repository: mesos > > > Description > ------- > > Updated `ROOT_CGROUPS_CFS_Big_Quota` to test a CFS quota with decimal. > > > Diffs > ----- > > src/tests/containerizer/cgroups_tests.cpp > 821c7860ab424bcfadbf597902e046bce9ba3044 > > Diff: https://reviews.apache.org/r/51794/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >