Hi dev, If there are no particular concerns, I will send out a vote email.
Thanks, Heesung On Tue, Jan 3, 2023 at 10:28 AM Heesung Sohn <heesung.s...@streamnative.io> wrote: > Hi Pulsar dev, > > I raised a PIP for the default config change(change the > aggregatePublisherStatsByProducerName config default to true) as > requested. > PIP: https://github.com/apache/pulsar/issues/19125 > > I guess we can continue to use this discussion thread for this PIP as well. > > Thank you, > Heesung > > > On Fri, Dec 9, 2022 at 2:12 PM Heesung Sohn <heesung.s...@streamnative.io> > wrote: > >> Hi, >> >> I raised a PR to the master branch here, >> https://github.com/apache/pulsar/pull/18807. >> >> PTAL. >> >> Thank you, >> Heesung >> >> On Wed, Nov 30, 2022 at 4:06 AM Enrico Olivelli <eolive...@gmail.com> >> wrote: >> >>> Heesung, >>> I also agree that we must preserve compatibility, if we want to pick >>> this change to released versions. >>> For a new major version we COULD break compatibility only as a last >>> resort. >>> >>> If we have a way to improve the code and use less resources, I believe >>> that it is the best way. >>> >>> Enrico >>> >>> Il giorno mer 30 nov 2022 alle ore 11:56 Heesung Sohn >>> <heesung.s...@streamnative.io.invalid> ha scritto: >>> > >>> > Hi, >>> > >>> > I raised this PR to my local fork to sync our direction. >>> > >>> > https://github.com/heesung-sn/pulsar/pull/15/files >>> > >>> > Please let me know if this direction looks good. Then, I will proceed >>> to >>> > raise an official PR. >>> > >>> > >>> > Thanks, >>> > Heesung >>> >>