heesung-sn commented on PR #18807: URL: https://github.com/apache/pulsar/pull/18807#issuecomment-1368061496
> Is it better to split it into 2 PRs? One is to fix the aggregation issue, another change the default value. > > So that we can ship the fix to release branches, and the default configuration change only applies to the master branch (release in the next major version). I reverted the config default change in this PR. I raised another PR for the config default change here. https://github.com/apache/pulsar/pull/19114 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
