> On May 1, 2013, 10:41 p.m., Benjamin Hindman wrote: > > third_party/libprocess/include/process/limiter.hpp, line 83 > > <https://reviews.apache.org/r/10891/diff/1/?file=286794#file286794line83> > > > > This was my bad, shouldn't require CHECK. s/CHECK/assert/ and add > > include <assert.h> above.
Why not CHECK? - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10891/#review20032 ----------------------------------------------------------- On May 1, 2013, 9:57 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10891/ > ----------------------------------------------------------- > > (Updated May 1, 2013, 9:57 p.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Description > ------- > > BenH's patch. > > > Diffs > ----- > > third_party/libprocess/Makefile.am d9d9110cc75d71642e621097bbc8bdcb88fdc0f8 > third_party/libprocess/include/process/limiter.hpp PRE-CREATION > third_party/libprocess/src/tests/process_tests.cpp > 8fbf19676c54eb0da6f8daa042de127608bcb00b > > Diff: https://reviews.apache.org/r/10891/diff/ > > > Testing > ------- > > sudo ./third_party/libprocess/tests --gtest_filter="*limiter*" --verbose > --gtest_repeat=1000 --gtest_break_on_failure > > > Thanks, > > Vinod Kone > >
