Looks there is no objection, I will start the official vote for PIP-122 Regards, Hang
Michael Marshall <mmarsh...@apache.org> 于2022年1月8日周六 07:44写道： > > Hi Frank, > > You can configure the Load Shedding Strategy in the `broker.conf` file > using `loadBalancerLoadSheddingStrategy`. > > The configuration is missing from the document you referenced--it > should be there, though. > > There was a recent PR to add documentation describing the different > configuration options: [0]. Those docs won't go live until 2.10.0 is > released. We could probably add parts of that documentation to the > historical docs, since the Shedding Strategy has existed for some > time. > > - Michael > > [0] https://github.com/apache/pulsar/pull/12902/files > > On Fri, Jan 7, 2022 at 1:29 PM Frank Kelly <fke...@cogitocorp.com> wrote: > > > > This is very cool. > > > > How is `ThresholdShedder` chosen - I don't see if referenced here? > > https://pulsar.apache.org/docs/en/reference-configuration/ > > > > -F > > > > On Thu, Jan 6, 2022 at 7:45 PM Sijie Guo <guosi...@gmail.com> wrote: > > > > > +1 ThresholdShedder is a much better policy > > > > > > On Tue, Jan 4, 2022 at 6:24 AM Hang Chen <chenh...@apache.org> wrote: > > > > > > > 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` > > > > > > >
