----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34308/#review84329 -----------------------------------------------------------
LGTM - want to add a test in src/tests/resources_tests.cpp too? - Niklas Nielsen On May 18, 2015, 1:49 p.m., Ian Downes wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34308/ > ----------------------------------------------------------- > > (Updated May 18, 2015, 1:49 p.m.) > > > Review request for mesos and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Filter revocable resources. > > > Diffs > ----- > > include/mesos/resources.hpp 1e98c13fe8075b14454f7899b98006fdaf88f484 > src/common/resources.cpp 92b9e7f603111123e0f7cf69c42e712468b631f3 > > Diff: https://reviews.apache.org/r/34308/diff/ > > > Testing > ------- > > > Thanks, > > Ian Downes > >