----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34432/#review84885 -----------------------------------------------------------
src/slave/containerizer/isolators/network/port_mapping.hpp <https://reviews.apache.org/r/34432/#comment136331> Please state why they are exposed. src/slave/containerizer/isolators/network/port_mapping.cpp <https://reviews.apache.org/r/34432/#comment136332> Why not be consistent with the other "const string" inlines? - Ian Downes On May 21, 2015, 4:32 p.m., Paul Brett wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34432/ > ----------------------------------------------------------- > > (Updated May 21, 2015, 4:32 p.m.) > > > Review request for mesos, Chi Zhang, Ian Downes, Jie Yu, and Cong Wang. > > > Bugs: MESOS-2754 > https://issues.apache.org/jira/browse/MESOS-2754 > > > Repository: mesos > > > Description > ------- > > Remove duplicate constant string references to mesos-containerizer, > net_tcp_rtt_... > > > Diffs > ----- > > src/slave/containerizer/isolators/network/port_mapping.hpp > c72fb47f60f40cda8d84a10497b9133f83cf018e > src/slave/containerizer/isolators/network/port_mapping.cpp > 49e983edab598e2ac487bb488fdd12840a9e7dfc > src/slave/containerizer/mesos/containerizer.hpp > 3e18617b0dbac58176bfd41dc550898eb6a4a79e > src/slave/containerizer/mesos/containerizer.cpp > 696e359de66305512eedf8e269543fafa21f4bc3 > src/tests/port_mapping_tests.cpp b8c2db6d0a02f79d38a21c227575299880980502 > > Diff: https://reviews.apache.org/r/34432/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Paul Brett > >