----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47222/#review132609 -----------------------------------------------------------
src/master/quota_handler.cpp (line 491) <https://reviews.apache.org/r/47222/#comment196868> Please advise if we should define "/quota" as a member const of `QuotaHandler` class. - Zhitao Li On May 11, 2016, 6:14 a.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47222/ > ----------------------------------------------------------- > > (Updated May 11, 2016, 6:14 a.m.) > > > Review request for mesos, Adam B and Alexander Rukletsov. > > > Bugs: MESOS-5336 > https://issues.apache.org/jira/browse/MESOS-5336 > > > Repository: mesos > > > Description > ------- > > Add authorization to /quota endpoint. > > > Diffs > ----- > > src/master/http.cpp a0d67671c89b9794e721dc3ba012acd3bdc447e3 > src/master/master.hpp 3e55114ee7866e06513071e86e15608099dae052 > src/master/quota_handler.cpp 6590519d8440f352f5bf00fda805414a5aad725c > src/tests/master_quota_tests.cpp 9ce7e2e481706c74f080d54b6f58d57874a27e64 > > Diff: https://reviews.apache.org/r/47222/diff/ > > > Testing > ------- > > Adding tests in MasterQuotaTest. > > > Thanks, > > Zhitao Li > >
