> On Dec. 13, 2016, 12:47 a.m., Anand Mazumdar wrote: > > src/tests/api_tests.cpp, line 1493 > > <https://reviews.apache.org/r/52639/diff/4/?file=1550804#file1550804line1493> > > > > Can you do this outside of this scope as it's common to all subsequent > > blocks? It would then be consistent to all the other tests in this file.
I think I will do this in a separate sweeping patch instead of this one. - Zhitao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52639/#review158935 ----------------------------------------------------------- On Dec. 14, 2016, 4:49 a.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52639/ > ----------------------------------------------------------- > > (Updated Dec. 14, 2016, 4:49 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 > ------- > > Added test for `recovered` AgentID and `AGENT_ADDED` after reregister. > > > Diffs > ----- > > src/tests/api_tests.cpp 82c0fc27e5e707adb73faeb26828a2ce3e3feb16 > src/tests/master_tests.cpp b3b5630899082a69119d2cccb6460766d1352963 > > Diff: https://reviews.apache.org/r/52639/diff/ > > > Testing > ------- > > > Thanks, > > Zhitao Li > >
