----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43816/#review120272 -----------------------------------------------------------
Ship it! Ship It! - Klaus Ma On Feb. 22, 2016, 3:11 p.m., Neil Conway wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43816/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2016, 3:11 p.m.) > > > Review request for mesos and Michael Park. > > > Bugs: MESOS-4731 > https://issues.apache.org/jira/browse/MESOS-4731 > > > Repository: mesos > > > Description > ------- > > Updated `/frameworks` master endpoint to use jsonify. > > > Diffs > ----- > > src/master/http.cpp ae6bc7852202480e58f579a5b48ab5b5e5ff9317 > > Diff: https://reviews.apache.org/r/43816/diff/ > > > Testing > ------- > > 1. make check > 2. Verified that after introducing a bug into the jsonify version of > `frameworks()`, `make check` fails (i.e., the test suite covers the > `/frameworks` endpoint). > 3. Compared output of `/frameworks` with old and new implementation on a test > Mesos installation to try to gauge correctness visually. > > > Thanks, > > Neil Conway > >