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

[Matteo Merli]

+1
--
Matteo Merli
<matteo.me...@gmail.com>

On Tue, Jan 11, 2022 at 12:05 PM Neng Lu <nl...@apache.org> wrote:
>
> +1 (non-binding)
>
> On Mon, Jan 10, 2022 at 12:39 AM Enrico Olivelli <eolive...@gmail.com>
> wrote:
>
> > +1 (binding)
> >
> > Enrico
> >
> > Il giorno lun 10 gen 2022 alle ore 07:47 Hang Chen
> > <chenh...@apache.org> 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 ThresholdShedder load balance policy since Pulsar 2.6.0 by
> > > https://github.com/apache/pulsar/pull/6772. It can resolve many load
> > > balance issues of `OverloadShedder` and works well in many Pulsar
> > > production clusters.
> > >
> > > In Pulsar 2.6.0, 2.7.0, 2.8.0 and 2.9.0, Pulsar's default load balance
> > > policy is `OverloadShedder`.
> > >
> > > I think it's a good time for 2.10 to change default load balance
> > > policy to `ThresholdShedder`, it will make throughput more balance
> > > between brokers.
> > >
> > > ### Proposed Changes
> > > In 2.10 release，for `broker.conf`, change
> > > `loadBalancerLoadSheddingStrategy` from
> > > `org.apache.pulsar.broker.loadbalance.impl.OverloadShedder` to
> > > `org.apache.pulsar.broker.loadbalance.impl.ThresholdShedder`
> >