Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-13 Thread Hang Chen
Thanks for your participation. Close this vote with 3 (+1) bindings and 5 (+1) non-bindings, 0 (-1). Regards, Hang PengHui Li 于2022年1月13日周四 11:03写道: > > +1 > > Penghui > > On Thu, Jan 13, 2022 at 7:54 AM Aloys Zhang wrote: > > > +1 > > > > Michael Marshall 于2022年1月12日周三 13:37写道: > > > > > +1

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-12 Thread PengHui Li
+1 Penghui On Thu, Jan 13, 2022 at 7:54 AM Aloys Zhang wrote: > +1 > > Michael Marshall 于2022年1月12日周三 13:37写道: > > > +1 - assuming we ensure that the `ThresholdShedder` has unit test > coverage. > > > > Thanks, > > Michael > > > > > > On Tue, Jan 11, 2022 at 9:53 PM r...@apache.org > > > wrot

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-12 Thread Aloys Zhang
+1 Michael Marshall 于2022年1月12日周三 13:37写道: > +1 - assuming we ensure that the `ThresholdShedder` has unit test coverage. > > Thanks, > Michael > > > On Tue, Jan 11, 2022 at 9:53 PM r...@apache.org > wrote: > > > > +1 (non-binding) > > > > -- > > > > Thanks > > Xiaolong Ran > > > > Haiting Jiang

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-11 Thread Michael Marshall
+1 - assuming we ensure that the `ThresholdShedder` has unit test coverage. Thanks, Michael On Tue, Jan 11, 2022 at 9:53 PM r...@apache.org wrote: > > +1 (non-binding) > > -- > > Thanks > Xiaolong Ran > > Haiting Jiang 于2022年1月12日周三 09:52写道: > > > +1 > > > > On 2022/01/10 06:47:44 Hang Chen w

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-11 Thread r...@apache.org
+1 (non-binding) -- Thanks Xiaolong Ran Haiting Jiang 于2022年1月12日周三 09:52写道: > +1 > > On 2022/01/10 06:47:44 Hang Chen wrote: > > This is the voting thread for PIP-122. It will stay open for at least 48 > > hours. > > > > https://github.com/apache/pulsar/issues/13340 > > > > Pasted below for q

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-11 Thread Haiting Jiang
+1 On 2022/01/10 06:47:44 Hang Chen wrote: > This is the voting thread for PIP-122. It will stay open for at least 48 > hours. > > https://github.com/apache/pulsar/issues/13340 > > Pasted below for quoting convenience. > > > > ### Motivation > The ThresholdShedder load balance policy sinc

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-11 Thread Matteo Merli
+1 -- Matteo Merli On Tue, Jan 11, 2022 at 12:05 PM Neng Lu wrote: > > +1 (non-binding) > > On Mon, Jan 10, 2022 at 12:39 AM Enrico Olivelli > wrote: > > > +1 (binding) > > > > Enrico > > > > Il giorno lun 10 gen 2022 alle ore 07:47 Hang Chen > > ha scritto: > > > > > > > > This is the voting

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-11 Thread Neng Lu
+1 (non-binding) On Mon, Jan 10, 2022 at 12:39 AM Enrico Olivelli wrote: > +1 (binding) > > Enrico > > Il giorno lun 10 gen 2022 alle ore 07:47 Hang Chen > ha scritto: > > > > > This is the voting thread for PIP-122. It will stay open for at least 48 > > hours. > > > > https://github.com/apache

Re: [VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-10 Thread Enrico Olivelli
+1 (binding) Enrico Il giorno lun 10 gen 2022 alle ore 07:47 Hang Chen ha scritto: > > This is the voting thread for PIP-122. It will stay open for at least 48 > hours. > > https://github.com/apache/pulsar/issues/13340 > > Pasted below for quoting convenience. > > > > ### Motivation > The

[VOTE] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

2022-01-09 Thread Hang Chen
This is the voting thread for PIP-122. It will stay open for at least 48 hours. https://github.com/apache/pulsar/issues/13340 Pasted below for quoting convenience. ### Motivation The ThresholdShedder load balance policy since Pulsar 2.6.0 by https://github.com/apache/pulsar/pull/6772. It c