---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 13, 2015, 10:57 a.m.)
Review request for mesos, Adam B,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 13, 2015, 1:20 p.m.)
Review request for mesos, Adam B, Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review95249
---
Ship it!
Fix minor typo, then ship it!
(We know that this is not
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 13, 2015, 7:08 p.m.)
Review request for mesos, Adam B, Alexander
On Aug. 13, 2015, 6:43 p.m., Timothy Chen wrote:
src/authorizer/authorizer.cpp, line 19
https://reviews.apache.org/r/36048/diff/18/?file=1039308#file1039308line19
Is this our new style guide rule too to put this up before all other
imports?
Otherwise I would have
On Aug. 13, 2015, 8:06 p.m., Till Toenshoff wrote:
Thanks for your patience Alexander - I know it has been a long journey - in
the end, I think things are in great shape minus some nits I just
discovered on my final pass.
Please note that I will commit while fixing the noted issues but
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review95266
---
Ship it!
Thanks for your patience Alexander - I know it has been a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 12, 2015, 4:56 p.m.)
Review request for mesos, Adam B, Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review95096
---
include/mesos/authorizer/authorizer.hpp (line 139)
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review95106
---
Good points Alex, can you leave comments on this in the code of to
On Aug. 12, 2015, 5:57 p.m., Timothy Chen wrote:
include/mesos/authorizer/authorizer.hpp, line 139
https://reviews.apache.org/r/36048/diff/16/?file=1038383#file1038383line139
Space between operator and
The rule changed, we no longer put spaces between operators. Check commit
On Aug. 12, 2015, 6:04 a.m., Bernd Mathiske wrote:
include/mesos/authorizer/authorizer.hpp, line 49
https://reviews.apache.org/r/36048/diff/15/?file=1036353#file1036353line49
If this is only relevant for the default implementation, then this
method should be declared there, not
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review95079
---
include/mesos/authorizer/authorizer.hpp (line 48)
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 12, 2015, 3:52 p.m.)
Review request for mesos, Adam B, Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 10, 2015, 4:35 p.m.)
Review request for mesos, Adam B, Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 6, 2015, 1:59 p.m.)
Review request for mesos, Adam B, Alexander
On Aug. 3, 2015, 4:20 p.m., Bernd Mathiske wrote:
include/mesos/authorizer/authorizer.proto, line 28
https://reviews.apache.org/r/36048/diff/11/?file=1017127#file1017127line28
What is local authorization? Link or explanation, please!
Added the sentence `(See authorization.md file
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 5, 2015, 11:04 a.m.)
Review request for mesos, Adam B, Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated Aug. 4, 2015, 4:13 p.m.)
Review request for mesos, Adam B, Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review94059
---
include/mesos/authorizer/authorizer.hpp (line 83)
On Aug. 3, 2015, 12:36 p.m., Kapil Arya wrote:
I have a stupid question. Why shouldn't the authorizer.{hpp,proto} file be
placed inside `include/mesos/master/` instead since the authorizer is only
for the master. Does it make sense to do that? If it does, then should we
also move
On Aug. 3, 2015, 12:36 p.m., Kapil Arya wrote:
I have a stupid question. Why shouldn't the authorizer.{hpp,proto} file be
placed inside `include/mesos/master/` instead since the authorizer is only
for the master. Does it make sense to do that? If it does, then should we
also move
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review93907
---
include/mesos/authorizer/authorizer.hpp (line 24)
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review93884
---
Ship it!
I think this looks ready for landing after fixing the
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review93912
---
src/authorizer/authorizer.cpp (line 209)
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review93900
---
include/mesos/authorizer/authorizer.hpp (line 34)
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/#review93951
---
I have a stupid question. Why shouldn't the authorizer.{hpp,proto}
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 20, 2015, 11 p.m.)
Review request for mesos, Adam B, Alexander
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 7, 2015, 9:33 a.m.)
Review request for mesos, Adam B and Till
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 6, 2015, 2:38 p.m.)
Review request for mesos, Adam B and Till
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 6, 2015, 5:41 p.m.)
Review request for mesos, Adam B and Till
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 6, 2015, 4:18 p.m.)
Review request for mesos, Adam B and Till
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 3, 2015, 3:15 p.m.)
Review request for mesos and Till Toenshoff.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 3, 2015, 9:49 a.m.)
Review request for mesos and Till Toenshoff.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36048/
---
(Updated July 3, 2015, 9:56 a.m.)
Review request for mesos and Till Toenshoff.
35 matches
Mail list logo