> On Nov. 22, 2015, 6:07 p.m., Joris Van Remoortere wrote: > > src/tests/registrar_tests.cpp, line 687 > > <https://reviews.apache.org/r/40400/diff/2/?file=1128834#file1128834line687> > > > > Let's use full words. `Alternate`, or we could enumerate them.
The reason why I didin't want to enumerate is to avoid potential confusion, because there are `quota1` and `quota2`, which are not directly tighed to `quotaResources1` and `quotaResources2`. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40400/#review107510 ----------------------------------------------------------- On Nov. 22, 2015, 5:02 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40400/ > ----------------------------------------------------------- > > (Updated Nov. 22, 2015, 5:02 p.m.) > > > Review request for mesos, Bernd Mathiske, Joerg Schad, Joris Van Remoortere, > and Joseph Wu. > > > Bugs: MESOS-3165 > https://issues.apache.org/jira/browse/MESOS-3165 > > > Repository: mesos > > > Description > ------- > > Follow up of https://reviews.apache.org/r/39983 > > > Diffs > ----- > > src/tests/registrar_tests.cpp f4ed01e9f37f8291921b67f7fb2f54839812c8b1 > > Diff: https://reviews.apache.org/r/40400/diff/ > > > Testing > ------- > > make check (Mac OS X 10.10.4) > > > Thanks, > > Alexander Rukletsov > >
