Yes, that sounds like a good idea. Updated the wiki for KIP-73. Thanks,
Jun On Fri, Oct 21, 2016 at 4:26 PM, Joel Koshy <jjkosh...@gmail.com> wrote: > Thanks for catching that and the fix as well. Makes sense to me. > > We should consider adding an "amendments" section to KIPs - perhaps just a > link to KAFKA-4313 would suffice in this case. > > Thanks, > > Joel > > On Wed, Oct 19, 2016 at 7:12 PM, Jun Rao <j...@confluent.io> wrote: > > > Hi, > > > > While testing KIP-73, we found an issue described in > > https://issues.apache.org/jira/browse/KAFKA-4313. Basically, when there > > are > > mixed high-volume and low-volume partitions, when replication throttling > is > > specified, ISRs for those low volume partitions could thrash. KAFKA-4313 > > fixes this issue by avoiding throttling those replicas in the throttled > > replica list that are already in sync. Those in-sync replicas traffic > will > > still be accounted for the throttled traffic though. Just want to bring > > this up since it slightly changes the behavior described in the KIP. If > > anyone has concerns on this, please comment on the jira. > > > > Thanks, > > > > Jun > > > > On Tue, Aug 23, 2016 at 3:25 PM, Ismael Juma <ism...@juma.me.uk> wrote: > > > > > For the record, there were 4 binding +1s. > > > > > > Ismael > > > > > > On Tue, Aug 23, 2016 at 11:16 PM, Ben Stopford <b...@confluent.io> > wrote: > > > > > > > Thanks everyone. It looks like this KIP has now been accepted. > > > > > > > > There is a corresponding PR <https://github.com/apache/ > kafka/pull/1776 > > > > > > > for the implementation also. > > > > > > > > All the best > > > > > > > > B > > > > > > > > > > > > > On 23 Aug 2016, at 22:39, Joel Koshy <jjkosh...@gmail.com> wrote: > > > > > > > > > > +1 > > > > > (sent some very minor edits to you off-thread) > > > > > > > > > > On Fri, Aug 19, 2016 at 1:21 AM, Ben Stopford <b...@confluent.io> > > > wrote: > > > > > > > > > >> I’d like to initiate the voting process for KIP-73: > > > > >> https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > > >> 73+Replication+Quotas <https://cwiki.apache.org/ > > > > >> confluence/display/KAFKA/KIP-73+Replication+Quotas> > > > > >> > > > > >> Ben > > > > > > > > > > > > > >