----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52873/#review152904 -----------------------------------------------------------
Fix it, then Ship it! src/health-check/health_checker.hpp (lines 142 - 143) <https://reviews.apache.org/r/52873/#comment222091> `clone` looks not an internal state here? How about just remove this comment? - haosdent huang On Oct. 14, 2016, 12:45 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52873/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2016, 12:45 p.m.) > > > Review request for mesos, Gastón Kleiman and haosdent huang. > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/health-check/health_checker.hpp > 392b4d5bd1e5831994b9366c1eb5a2911e19860f > src/health-check/health_checker.cpp > 96ae1a733ff3d211b84d0893b4603873af1c89f0 > > Diff: https://reviews.apache.org/r/52873/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rukletsov > >
