> On Oct. 22, 2015, 3:08 a.m., Guangya Liu wrote: > > src/master/http.cpp, line 839 > > <https://reviews.apache.org/r/36913/diff/19/?file=1102864#file1102864line839> > > > > what about "return quotaHandler.create(request)" > > Joerg Schad wrote: > Wanted to make names consistent. > @alex: any preference?
Yes. Naming is hard, we had a long offline discussion about these names and decided the following: - Be consistent. Though there are subtle differences between operations depending on the stage in the process, they are minor and will most probably lead to confusion. - `set` instead of `add`, `request`, or `create`. You cannot call `set` multiple times, that's why `add` is not good. `request` is too vague, you can request anything. `create` carries the meaning of a creation act, which is not the focus we would like to have. An important message to deliver is that `set` my change the allocation behaviour for the role significantly. - `update` (not in MVP). - `remove` instead of `release`, `destroy`, or `delete`. `destroy` and `delete` focus on scrapping an object, which is not the most important action in this case. We would like to put accent on the fact that there is no more quota associated with the role. `release` for me implies possession, moreover, it doesn't play nice with `set`. - CRUD naming is a reasonable alternative (because it's a standard), however we decided to opt for more specific naming in a hope it will help understanding quota implications better. - This is all about internal naming, the operator API uses standard HTTP verbs for these actions. > On Oct. 22, 2015, 3:08 a.m., Guangya Liu wrote: > > src/master/http.cpp, line 843 > > <https://reviews.apache.org/r/36913/diff/19/?file=1102864#file1102864line843> > > > > What about return quotaHandler.delete(request); > > > > It is better that the operation API can map to CRUD > > Joerg Schad wrote: > We wanted to make the name consistent across all calls. @alex: Any > preference? Yep, see above. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36913/#review103518 ----------------------------------------------------------- On Oct. 22, 2015, 1:03 a.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36913/ > ----------------------------------------------------------- > > (Updated Oct. 22, 2015, 1:03 a.m.) > > > Review request for mesos, Alexander Rukletsov, Bernd Mathiske, Alex Clemmer, > and Joris Van Remoortere. > > > Bugs: MESOS-3073 > https://issues.apache.org/jira/browse/MESOS-3073 > > > Repository: mesos > > > Description > ------- > > Added /quota HTTP Endpoint for Quota handling. > > > Diffs > ----- > > src/CMakeLists.txt e6169a0e3ad34dd0e4c3430a6532bd48c4bd04fd > src/Makefile.am 98cbafc134ec388a176d50172912fbfdf9f5bfa3 > src/master/http.cpp 093f79384916dc08b32b70d3614e0ff314825c42 > src/master/master.hpp e7b16fdd21a8caa77a39956a8520cf1381186598 > src/master/master.cpp 0981428f5a93242d6fb3f68fb93d5f8ee6178ace > src/master/quota_handler.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/36913/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Joerg Schad > >