Re: Review Request 21352: Fix regression causing scheduling rate limiter to not be honored.

2014-05-13 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21352/#review42858 --- Ship it! Ship It! - Kevin Sweeney On May 13, 2014, 11:13 a.m., B

Re: Review Request 21352: Fix regression causing scheduling rate limiter to not be honored.

2014-05-13 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21352/#review42852 --- config/findbugs/excludeFilter.xml

Re: Review Request 21352: Fix regression causing scheduling rate limiter to not be honored.

2014-05-13 Thread Kevin Sweeney
> On May 13, 2014, 11:05 a.m., Kevin Sweeney wrote: > > config/findbugs/excludeFilter.xml, line 9 > > > > > > You should be able to add a java namespace to the .thrift file here > > (one already exists for python) > >

Re: Review Request 21352: Fix regression causing scheduling rate limiter to not be honored.

2014-05-13 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21352/ --- (Updated May 13, 2014, 6:13 p.m.) Review request for Aurora, David McLaughlin a

Re: Review Request 21352: Fix regression causing scheduling rate limiter to not be honored.

2014-05-13 Thread Bill Farner
> On May 13, 2014, 6:05 p.m., Kevin Sweeney wrote: > > config/findbugs/excludeFilter.xml, line 9 > > > > > > You should be able to add a java namespace to the .thrift file here > > (one already exists for python) Punti

Re: Review Request 21352: Fix regression causing scheduling rate limiter to not be honored.

2014-05-13 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21352/#review42834 --- Ship it! Ship It! - Maxim Khutornenko On May 13, 2014, 12:06 a.m

Re: Review Request 21352: Fix regression causing scheduling rate limiter to not be honored.

2014-05-12 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21352/#review42792 --- Ship it! lgtm. - David McLaughlin On May 13, 2014, 12:06 a.m.,