----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48593/#review140954 -----------------------------------------------------------
src/common/values.cpp (line 270) <https://reviews.apache.org/r/48593/#comment206329> Period to the end. src/common/values.cpp (line 276) <https://reviews.apache.org/r/48593/#comment206331> what about for(int i = 0; i < ranges.range_size(); i++) { and remove L273 src/common/values.cpp (line 285) <https://reviews.apache.org/r/48593/#comment206330> Period to the end. src/tests/values_tests.cpp (lines 33 - 34) <https://reviews.apache.org/r/48593/#comment206332> a new line here src/tests/values_tests.cpp (line 316) <https://reviews.apache.org/r/48593/#comment206333> s/Unit test/Test src/tests/values_tests.cpp (line 338) <https://reviews.apache.org/r/48593/#comment206351> s/{[1,2), [3-6), [7,9)}/{[1,2)[3,6)[7,9)} src/tests/values_tests.cpp (line 341) <https://reviews.apache.org/r/48593/#comment206355> s/1U/1 ditto for others src/tests/values_tests.cpp (line 354) <https://reviews.apache.org/r/48593/#comment206334> s/Unit test/Test src/tests/values_tests.cpp (line 360) <https://reviews.apache.org/r/48593/#comment206352> s/{[1-2), [3-5), [7-10)}/{[1,2)[3-5)[7,10)} - Guangya Liu On 六月 15, 2016, 3:39 a.m., Yanyan Hu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48593/ > ----------------------------------------------------------- > > (Updated 六月 15, 2016, 3:39 a.m.) > > > Review request for mesos, Guangya Liu and Joris Van Remoortere. > > > Bugs: MESOS-5425 > https://issues.apache.org/jira/browse/MESOS-5425 > > > Repository: mesos > > > Description > ------- > > This patch reimplement Ranges subtraction using > IntervalSet data type: Ranges values will be > converted to IntervalSet values for subtraction > and the result will be converted back to Ranges > after subtraction is done. This change is for > fixing jira MESOS-5425. > > > Diffs > ----- > > src/common/values.cpp 587cb68551d438621e215953e89818b623b7f71b > src/tests/values_tests.cpp 929861549e3155c33966896f817f9bf9e6d14354 > > Diff: https://reviews.apache.org/r/48593/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Yanyan Hu > >