> On Dec. 12, 2016, 10:11 p.m., Anand Mazumdar wrote: > > src/master/http.cpp, line 158 > > <https://reviews.apache.org/r/52765/diff/5/?file=1551621#file1551621line158> > > > > Any reason for not adding `port` here?
It was never added in the past. I will add it in this patch. - Zhitao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52765/#review158902 ----------------------------------------------------------- On Dec. 14, 2016, 4:48 a.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52765/ > ----------------------------------------------------------- > > (Updated Dec. 14, 2016, 4:48 a.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 `/state` and `/slaves` endpoints. > > > Diffs > ----- > > src/master/http.cpp d52806dcf8e4d64ebb98e191a01408c0fcae17ac > src/tests/master_tests.cpp b3b5630899082a69119d2cccb6460766d1352963 > > Diff: https://reviews.apache.org/r/52765/diff/ > > > Testing > ------- > > > Thanks, > > Zhitao Li > >
