----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63794/#review191127 -----------------------------------------------------------
src/checks/health_checker.cpp Lines 266-273 (patched) <https://reviews.apache.org/r/63794/#comment268734> Hm, maybe this is more readable? ``` if ((healthCheck.type() == HealthCheck.HTTP && healthCheck.http().protocol() == NetworkInfo::IPv6) || (healthCheck.type() == HealthCheck.TCP && healthCheck.tcp().protocol() == NetworkInfo::IPv6)) { ipv6 = true; } ``` - Alexander Rukletsov On Nov. 14, 2017, 1:27 p.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63794/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2017, 1:27 p.m.) > > > Review request for mesos, Alexander Rukletsov and Avinash sridharan. > > > Bugs: MESOS-8050 > https://issues.apache.org/jira/browse/MESOS-8050 > > > Repository: mesos > > > Description > ------- > > Added a new member field `ipv6` to the `CheckerProcess` class. > > > Diffs > ----- > > src/checks/checker_process.hpp 5d0c36dd3e1edf68527fc109d8c0708f09b7e12f > src/checks/checker_process.cpp 7985e8d4dc6115f81a4daef0ce4eee67f4233c9f > src/checks/health_checker.cpp d4bda6ed1747ae4c970619bbfa336321aeea52ea > > > Diff: https://reviews.apache.org/r/63794/diff/1/ > > > Testing > ------- > > > Thanks, > > Qian Zhang > >
