----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11331/#review21541 -----------------------------------------------------------
third_party/libprocess/third_party/stout/include/stout/duration.hpp <https://reviews.apache.org/r/11331/#comment44583> Well.. depends how one defines elegance, I had a hard time following this logic ;) For example, this code has the following nests: for { if { if { if { } } break; } } and I need to understand the buckets, the sentinels, how the mappings work, etc. That is, it's hard for me to quickly look at this code and determine it's correctness. - Ben Mahler On May 22, 2013, 7:08 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11331/ > ----------------------------------------------------------- > > (Updated May 22, 2013, 7:08 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > - Currently when the result isn't a whole number, we go one-level down in > terms of the unit of time and see if we can get a whole number there. If so, > use the lower unit; if not, use the higher unit. > > > This addresses bug MESOS-477. > https://issues.apache.org/jira/browse/MESOS-477 > > > Diffs > ----- > > third_party/libprocess/third_party/stout/include/stout/duration.hpp > c943ab196871cf4c55f2a2dce5ef85ffe3a22577 > third_party/libprocess/third_party/stout/tests/duration_tests.cpp > 421615aa510f540ccfc0413d3252bbdedd70f7da > > Diff: https://reviews.apache.org/r/11331/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jiang Yan Xu > >
