----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58095/#review171674 -----------------------------------------------------------
src/master/http.cpp Lines 3504-3507 (patched) <https://reviews.apache.org/r/58095/#comment244692> Is there a better way to write empty array to json? - Jay Guo On April 12, 2017, 1:35 p.m., Jay Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58095/ > ----------------------------------------------------------- > > (Updated April 12, 2017, 1:35 p.m.) > > > Review request for mesos, Adam B, Alexander Rojas, Benjamin Mahler, and > Michael Park. > > > Bugs: MESOS-4732 and MESOS-7260 > https://issues.apache.org/jira/browse/MESOS-4732 > https://issues.apache.org/jira/browse/MESOS-7260 > > > Repository: mesos > > > Description > ------- > > Instead of generating JSON object, `Master::Http::roles()` now > leverages `jsonify` to compute output. Also approver is taken > out from its continuation function. This is for easier and cleaner > implementation of framework authorization in /roles endpoint, > see MESOS-7260. > > > Diffs > ----- > > src/master/http.cpp 0e84d363b346de1ef05101dac066c2f0cf31609d > src/master/master.hpp d537933d0b467a6f9996951c601b31338bb9d034 > > > Diff: https://reviews.apache.org/r/58095/diff/3/ > > > Testing > ------- > > no functional changes > make check > > > Thanks, > > Jay Guo > >
