----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28262/#review62271 -----------------------------------------------------------
Ship it! Ship It! src/proxy/proxy.hpp <https://reviews.apache.org/r/28262/#comment104294> do you think the process::Node should be passed by const reference? src/proxy/proxy.cpp <https://reviews.apache.org/r/28262/#comment104295> do you need to do this? process will go out of scope and be destroyed anyway on the next line. src/tests/proxy_tests.cpp <https://reviews.apache.org/r/28262/#comment104296> BWAAAAAAAAAA - Dominic Hamon On Nov. 19, 2014, 3:33 p.m., Cody Maloney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28262/ > ----------------------------------------------------------- > > (Updated Nov. 19, 2014, 3:33 p.m.) > > > Review request for mesos, Benjamin Hindman and Dominic Hamon. > > > Bugs: MESOS-2131 > https://issues.apache.org/jira/browse/MESOS-2131 > > > Repository: mesos-git > > > Description > ------- > > Attach/detach remote servers locally > > /proxy/debug.json lists attached endpoints. > /proxy/go/{name} forwards the rest of the path + query parameters to the > remote host. > > > Diffs > ----- > > src/Makefile.am 0fe7dd067b80cfe802201482eed63a30ac8dcbd4 > src/proxy/proxy.hpp PRE-CREATION > src/proxy/proxy.cpp PRE-CREATION > src/tests/proxy_tests.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/28262/diff/ > > > Testing > ------- > > make check on ArchLinux, make distcheck ubuntu 14.04 > > > Thanks, > > Cody Maloney > >
