HQebupt commented on pull request #13563: URL: https://github.com/apache/pulsar/pull/13563#issuecomment-1002845526
> This is interesting. > > Do you have some numbers about the impact of this change? > > As this is broken and current property is not taking effect... Why not removing the property at all and let users configure it? > > I am not sure about the actual benefit or the impact of restoring that option > > @merlimat @MMirelli @dave2wave Hi Enrico, I did not see any performance impact. I agree with you. And we can use the default value (`io.netty.recycler.maxCapacityPerThread=4096`) explicitly in that it keeps the same configuration as before in Pulsar. How about it? -- 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]
