----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37500/#review96478 -----------------------------------------------------------
src/master/master.cpp (line 1920) <https://reviews.apache.org/r/37500/#comment151874> I think that the logic in void Master::_subscribe( const UPID& from, const scheduler::Call::Subscribe& subscribe, const Future<bool>& authorized) should also be udpated? - Guangya Liu On Aug. 25, 2015, 8:25 p.m., Aditi Dixit wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37500/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2015, 8:25 p.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-3240 > https://issues.apache.org/jira/browse/MESOS-3240 > > > Repository: mesos > > > Description > ------- > > Added user to master's state, slave's state (not exposed) and updated user in > all slaves that have the registered framework. Added the test too. > > > Diffs > ----- > > src/master/http.cpp 37d76ee72f6a037f551bf2609e9393e16b496e44 > src/master/master.hpp 36c67599ef2c470da8d95f2caf926a154342d2cc > src/master/master.cpp 95207d24db0aa052eb70c4cc7eb75d0611c365cf > src/messages/messages.proto 8977d8e0f3b16003128b6b9cab556a7b224f083c > src/slave/slave.hpp 09172f7ed547049b3bd169b3db9be94e14f6bc39 > src/slave/slave.cpp 2a99abc00c525a93508d38e74d351d2f36572d86 > src/tests/fault_tolerance_tests.cpp > 89cb18be96cd60fb77fbcc4acd08cebdcf1ba075 > src/tests/master_tests.cpp 8a6b98b9f59ead20f537eb60b5084feed069a5b1 > src/tests/slave_recovery_tests.cpp e1392a2235ff51dac7a3cb7cd3e8edf8406864fc > src/tests/slave_tests.cpp d55e9dd4f4eb84a8fda85439e31a38e70890b377 > > Diff: https://reviews.apache.org/r/37500/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Aditi Dixit > >
