----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50062/#review143080 -----------------------------------------------------------
Bad patch! Reviews applied: [50062, 50172] Failed command: ./support/apply-review.sh -n -r 50062 Error: 2016-07-21 12:52:43 URL:https://reviews.apache.org/r/50062/diff/raw/ [5006/5006] -> "50062.patch" [1] Total errors found: 0 Checking 1 files Error: No line in the commit message summary may exceed 72 characters. Full log: https://builds.apache.org/job/mesos-reviewbot/14407/console - Mesos ReviewBot On July 21, 2016, 2:07 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50062/ > ----------------------------------------------------------- > > (Updated July 21, 2016, 2:07 a.m.) > > > Review request for mesos, Benjamin Mahler and Jiang Yan Xu. > > > Repository: mesos > > > Description > ------- > > This patch refactored the helper function of `makePortRanges` as this: > > 1) Renamed `makePortRanges` to `makePorts`, this function creates a > "ports(*)" resource for the given ranges. > 2) Added a new helper function `fragment`, it helps to fragments the > given range bounds into a number of subranges. > 3) Keep `makeRange` unchanged but updated its parameter using `uint64_t`. > > > Diffs > ----- > > src/tests/hierarchical_allocator_tests.cpp > befb94adb15e051abae135c7697538bd0cf83852 > > Diff: https://reviews.apache.org/r/50062/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Guangya Liu > >
