> On July 26, 2015, 9:37 p.m., Jojy Varghese wrote: > > src/master/http.cpp, line 339 > > <https://reviews.apache.org/r/36720/diff/3/?file=1021869#file1021869line339> > > > > complexity of a function is measured by the nested if condition in it. > > Maybe we accomplish the same using simple small functions ?
Yes, we intend to move these in the near future. To add a bit of context here, the validation logic would be handled in MESOS-2497. I added the band-aid logic here to prevent the master from crashing for now. Here is the TODO mentioned in the code: // Move this to unmarshal(...) method. - Anand ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36720/#review93060 ----------------------------------------------------------- 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 > >
