----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22777/#review46284 -----------------------------------------------------------
include/mesos/mesos.proto <https://reviews.apache.org/r/22777/#comment81593> this API is counter-intuitive. I would expect, because limits are set per-principle, that this default qps would also be applied per-principle. Is there a technical reason why we can't set a limiter up for every framework we encounter with the qps set to a limit (if in limits) or the default (if not)? - Dominic Hamon On June 19, 2014, 12:30 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22777/ > ----------------------------------------------------------- > > (Updated June 19, 2014, 12:30 p.m.) > > > Review request for mesos, Ben Mahler and Vinod Kone. > > > Bugs: MESOS-1492 > https://issues.apache.org/jira/browse/MESOS-1492 > > > Repository: mesos-git > > > Description > ------- > > In current previous design and implementation of framework rate limiting, > Master only throttles frameworks specified explicitly in RateLimits. > Frameworks not specified are not throttled. > > We can add an {{optional double aggregate_default_qps}} to RateLimits > protobuf and throttle all of the unspecified frameworks together using one > RateLimiter at the {{aggregate_default_qps}} rate. > > This allows the Mesos cluster operator to safeguard the Master from > unexpected frameworks and it's optional. > > > Diffs > ----- > > include/mesos/mesos.proto 2f6be059d3626663b73417863b42505c36f0446f > src/master/flags.hpp 47bb0dc04504fc847cc02fa97eb0d045a8835c9e > src/master/master.hpp 2844446e2674df6e11f4c2915ed324fcc103532c > src/master/master.cpp 888657dd4bc50085882382908e3c48ccb857c621 > src/tests/rate_limiting_tests.cpp 9a544618e70e7ce3091fa303597eb1bb3d161bf9 > > Diff: https://reviews.apache.org/r/22777/diff/ > > > Testing > ------- > > Added/modified tests for the default limiter. > make check. > > > Thanks, > > Jiang Yan Xu > >
