----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15288/#review29748 -----------------------------------------------------------
Ship it! src/master/http.cpp <https://reviews.apache.org/r/15288/#comment57218> YOu don't need to take the address of master.flags now right? src/slave/http.cpp <https://reviews.apache.org/r/15288/#comment57219> Ditto previous comment. - Benjamin Hindman On Nov. 27, 2013, 4:40 a.m., Niklas Nielsen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15288/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2013, 4:40 a.m.) > > > Review request for mesos, Benjamin Hindman and Ross Allen. > > > Repository: mesos-git > > > Description > ------- > > This patch adds flags field to master and slave state.json end-points which > expose all configuration flags as additional diagnostics. > > > Diffs > ----- > > src/master/http.cpp 218906a > src/slave/http.cpp fbc917b > > Diff: https://reviews.apache.org/r/15288/diff/ > > > Testing > ------- > > Functional testing of endpoints. > > make check > > > Thanks, > > Niklas Nielsen > >
