Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-05-02 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review131307 --- Ship it! Ship It! - Alexander Rukletsov On April 29, 2016, 2

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-29 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review131179 --- Patch looks great! Reviews applied: [46735, 46736] Passed comman

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-29 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review131100 --- Ship it! Ship It! - Neil Conway On April 29, 2016, 2:26 p.m.

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-29 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/ --- (Updated April 29, 2016, 2:26 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-29 Thread Neil Conway
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review131089 --- docs/endpoints/master/quota.md (line 29)

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-29 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/ --- (Updated April 29, 2016, 9:40 a.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-28 Thread Alexander Rukletsov
> On April 27, 2016, 1:17 p.m., Alexander Rukletsov wrote: > > src/master/http.cpp, lines 1272-1274 > > > > > > Any reason you wrap it differently from other entries? It also looks like we indent by spaces, can you

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-28 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review130892 --- Ship it! Ship It! - Alexander Rojas On April 27, 2016, 5:17

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-27 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review130800 --- Ship it! Ship It! - haosdent huang On April 27, 2016, 3:17 p

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-27 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/ --- (Updated April 27, 2016, 3:17 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review130798 --- Bad patch! Reviews applied: [46736, 46735] Failed command: ./sup

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-27 Thread Joerg Schad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/ --- (Updated April 27, 2016, 3:05 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-27 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review130777 --- Fix it, then Ship it! docs/endpoints/master/quota.md (line 29)

Re: Review Request 46736: Added authorization description to quota endpoint help.

2016-04-27 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46736/#review130776 --- Bad patch! Reviews applied: [46736, 46735] Failed command: ./sup