----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62197/#review185304 -----------------------------------------------------------
src/tests/mesos.hpp Line 590 (original), 590 (patched) <https://reviews.apache.org/r/62197/#comment261610> Pass by const ref instead. I'll fix this while committing. - Greg Mann On Sept. 12, 2017, 11:49 p.m., Gastón Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62197/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2017, 11:49 p.m.) > > > Review request for mesos, Benjamin Bannier, Greg Mann, and Joerg Schad. > > > Bugs: MESOS-7877 > https://issues.apache.org/jira/browse/MESOS-7877 > > > Repository: mesos > > > Description > ------- > > These new overloads make it possible to specify a framework ID, executor > resources as a proto, and the executor ID also as a proto. > > > Diffs > ----- > > src/tests/mesos.hpp 444c75763aea995708db0b17eafee2d22c912554 > > > Diff: https://reviews.apache.org/r/62197/diff/6/ > > > Testing > ------- > > `make check` still passes on GNU/Linux. > > > Thanks, > > Gastón Kleiman > >