Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-17 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 17, 2016, 6:10 a.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/#review138135 --- Ship it! Ship It! - Alexander Rukletsov On June 16, 2016,

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 11:03 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 10:47 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 10:47 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 10:34 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 10:20 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 10:10 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 9:45 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 9:02 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 8:43 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
> On May 30, 2016, 4:17 p.m., Joerg Schad wrote: > > src/authorizer/local/authorizer.cpp, line 59 > > > > > > Just curious: will we keep this around after the deprecation cycle? Left a comment to point out that this

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
> On May 30, 2016, 9:39 p.m., Alexander Rukletsov wrote: > > src/authorizer/local/authorizer.cpp, line 352 > > > > > > Introducing `approved()` looks like a general enhancement. Maybe do it > > in a separate

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
> On May 30, 2016, 9:39 p.m., Alexander Rukletsov wrote: > > src/authorizer/local/authorizer.cpp, lines 369-373 > > > > > > You make these now private. Is it on purpose? Do you want to do it in a > > separate patch

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-06-16 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- (Updated June 16, 2016, 5:59 p.m.) Review request for mesos, Alexander

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-05-30 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/#review135548 --- src/authorizer/local/authorizer.cpp (line 349)

Re: Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-05-30 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/#review135524 --- src/authorizer/local/authorizer.cpp (line 59)

Review Request 48038: Updated `LocalAuthorizer` to consolidate to the `UPDATE_QUOTA` action.

2016-05-30 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48038/ --- Review request for mesos, Alexander Rukletsov, Joris Van Remoortere, Vinod