----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22150/#review44573 -----------------------------------------------------------
Ship it! src/local/local.cpp <https://reviews.apache.org/r/22150/#comment78989> Do you need the temporary or can this just be: authorizer = authorizer_.get().release(); src/master/main.cpp <https://reviews.apache.org/r/22150/#comment78991> Add the option header include? src/master/main.cpp <https://reviews.apache.org/r/22150/#comment78990> Ditto here. src/master/master.cpp <https://reviews.apache.org/r/22150/#comment78994> Curious why you decided to add the 'owned' include even though there are no new Owned things in this change. At least we should include 'option' then too? src/master/master.cpp <https://reviews.apache.org/r/22150/#comment78992> "Authorization is enabled"? src/tests/cluster.hpp <https://reviews.apache.org/r/22150/#comment78993> Ditto here. - Ben Mahler On June 2, 2014, 6:01 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22150/ > ----------------------------------------------------------- > > (Updated June 2, 2014, 6:01 p.m.) > > > Review request for mesos and Ben Mahler. > > > Repository: mesos-git > > > Description > ------- > > Injected the Authorizer so that I can write more interesting tests. > > > Diffs > ----- > > src/local/local.cpp 5d26afffcdca0eb9d19499564a8edd2bf3dc1e66 > src/master/main.cpp 8ceaae61a732a8d68d19c8aee97c1fd93a595893 > src/master/master.hpp d4ef4bec7168179f2168e88d3727e50b0e2e68a1 > src/master/master.cpp 766a0e36a6e7a615e7b2974d9fee70bcef446719 > src/tests/cluster.hpp f4cc9a62cd0ca86cd87987d963abd951a377ddd1 > > Diff: https://reviews.apache.org/r/22150/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
