----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47558/#review135308 -----------------------------------------------------------
include/mesos/authorizer/authorizer.hpp (line 40) <https://reviews.apache.org/r/47558/#comment200326> ObjectAuthorizer include/mesos/authorizer/authorizer.hpp (line 63) <https://reviews.apache.org/r/47558/#comment200327> authorized src/authorizer/local/authorizer.cpp (line 174) <https://reviews.apache.org/r/47558/#comment200328> Individual lines src/authorizer/local/authorizer.cpp (lines 179 - 186) <https://reviews.apache.org/r/47558/#comment200331> indent src/master/http.cpp (line 206) <https://reviews.apache.org/r/47558/#comment200336> indent src/master/http.cpp (line 225) <https://reviews.apache.org/r/47558/#comment200333> indent src/master/http.cpp (line 233) <https://reviews.apache.org/r/47558/#comment200338> Move up - Joerg Schad On May 27, 2016, 3:48 a.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47558/ > ----------------------------------------------------------- > > (Updated May 27, 2016, 3:48 a.m.) > > > Review request for mesos, Adam B and Michael Park. > > > Bugs: MESOS-5403 > https://issues.apache.org/jira/browse/MESOS-5403 > > > Repository: mesos > > > Description > ------- > > With to goal to provide more efficient authorization > for multiple, potentially large objects we extend the > authorizer interface to support `ObjectAllower`. > Retrieving an ObjectAllower for a given action is an > asynchronous operation but following authorization > of multiple objects can be done synchronously without > copying the object. > NOTE: This implies that Authorizer Module writers > need to ensure an ObjectAllower will behave nicely > and especially not block. > > > Diffs > ----- > > include/mesos/authorizer/authorizer.hpp > ed5f9e73661e25a83722cf1e408ae61023cd4a21 > src/authorizer/local/authorizer.hpp > 61388454025211fd7d53e71a86983fd8479950b6 > src/authorizer/local/authorizer.cpp > dc53bc4374aea98b5ed41ade5617374d2447229b > src/master/http.cpp b36b439a1fa07c52146deff2b90728f92676ade3 > src/tests/mesos.hpp 79bf1ff16412ce2a510a9b75ab1ac91c1c182653 > src/tests/mesos.cpp 629135f0dc59346f0fcddb2cbe65ca5770fad34e > > Diff: https://reviews.apache.org/r/47558/diff/ > > > Testing > ------- > > tested entire chain. > > > Thanks, > > Joerg Schad > >
