----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52765/#review154091 -----------------------------------------------------------
src/master/http.cpp (lines 2652 - 2655) <https://reviews.apache.org/r/52765/#comment223598> You might want to create a helper overload function similar to the existing ones for `MasterInfo` etc. since `/slaves` endpoint would want to list the recovered agents too? I don't see it being added in this change. - Anand Mazumdar On Oct. 21, 2016, 6:40 p.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52765/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2016, 6:40 p.m.) > > > Review request for mesos, Anand Mazumdar, Xiaojian Huang, and Kunal Thakar. > > > Bugs: MESOS-6177 > https://issues.apache.org/jira/browse/MESOS-6177 > > > Repository: mesos > > > Description > ------- > > Populated `recovered_slaves` in master `/state` endpoint. > > > Diffs > ----- > > src/master/http.cpp 05d29906a73fb049c085abca05b75ec68c259d26 > > Diff: https://reviews.apache.org/r/52765/diff/ > > > Testing > ------- > > > Thanks, > > Zhitao Li > >
