----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9091/#review16068 -----------------------------------------------------------
third_party/libprocess/include/process/statistics.hpp <https://reviews.apache.org/r/9091/#comment34373> This comment is now out of date. third_party/libprocess/include/process/statistics.hpp <https://reviews.apache.org/r/9091/#comment34374> Update the comment with "... at the current clock time or at some specified time." third_party/libprocess/include/process/statistics.hpp <https://reviews.apache.org/r/9091/#comment34375> Need to bring in 'process/clock.hpp' now. Also, s/Seconds time/const Seconds& time/. third_party/libprocess/src/statistics.cpp <https://reviews.apache.org/r/9091/#comment34384> s/Seconds/const Seconds&/ third_party/libprocess/src/statistics.cpp <https://reviews.apache.org/r/9091/#comment34379> NICE cleanup! I like the macroness of it! third_party/libprocess/src/statistics.cpp <https://reviews.apache.org/r/9091/#comment34381> You can 'else if'. - Benjamin Hindman On Feb. 1, 2013, 1:11 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9091/ > ----------------------------------------------------------- > > (Updated Feb. 1, 2013, 1:11 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > This adds context into the Statistics model. Contexts provide a clean way to > logically group stats (e.g. contexts: "master", "slave", "monitor", > "libprocess", etc). > > > This addresses bug MESOS-324. > https://issues.apache.org/jira/browse/MESOS-324 > > > Diffs > ----- > > third_party/libprocess/include/process/statistics.hpp > 9e3041a6e2a8ef022eacacad00bc4d974a8e33c9 > third_party/libprocess/src/statistics.cpp > 2fe8af83c6c63a0fa8cb2e9636f9289f0e3d7f2f > third_party/libprocess/src/tests/statistics_tests.cpp > 0aaab3526618171c7cfbd11d40d614344bcbfd0a > > Diff: https://reviews.apache.org/r/9091/diff/ > > > Testing > ------- > > make check + libprocess make check > > > Thanks, > > Ben Mahler > >
