----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59481/#review175795 -----------------------------------------------------------
Ship it! I would find it helpful to have a summary of what this test is trying to do. I gather that it is demonstrating that even when a client is marked inactive, it's share of resources is persisted? - James Peach On May 23, 2017, 6:26 a.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59481/ > ----------------------------------------------------------- > > (Updated May 23, 2017, 6:26 a.m.) > > > Review request for mesos, Benjamin Mahler, James Peach, Michael Park, and > Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > This behavior has not changed, but was not covered by the existing > tests. > > > Diffs > ----- > > src/tests/sorter_tests.cpp 2389664df401a4579d7a7a218d552d1d1bec5536 > > > Diff: https://reviews.apache.org/r/59481/diff/1/ > > > Testing > ------- > > `make check` > > > Thanks, > > Neil Conway > >
