----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36720/#review93060 -----------------------------------------------------------
src/master/http.cpp (line 339) <https://reviews.apache.org/r/36720/#comment147323> complexity of a function is measured by the nested if condition in it. Maybe we accomplish the same using simple small functions ? - Jojy Varghese On July 25, 2015, 2:32 p.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36720/ > ----------------------------------------------------------- > > (Updated July 25, 2015, 2:32 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-2294 > https://issues.apache.org/jira/browse/MESOS-2294 > > > Repository: mesos > > > Description > ------- > > Split review out of r36318. This change adds the functionality of making a > http call for subscribe and the master responding with a subscribed event on > the persistent stream. > > > Diffs > ----- > > src/master/http.cpp 3a1598fad4db03e5f62fd4a6bd26b2bedeee4070 > src/master/master.hpp 827d0d599912b2936beb9615610f627f6c9a2d43 > src/master/master.cpp 613a011e205611702921179b6c436d62447e2dca > src/tests/http_api_tests.cpp 64bbeb6699171e85a5be293919ad9f32ded0ebac > > Diff: https://reviews.apache.org/r/36720/diff/ > > > Testing > ------- > > make check + test. > > > Thanks, > > Anand Mazumdar > >