lhotari commented on pull request #14320: URL: https://github.com/apache/pulsar/pull/14320#issuecomment-1042782213
> I think I need to explain more for the **important** term I used. IMO, a PR that could block a release during the release phase must match following rules: I haven't requested to block the release. > Let's look back to this PR. First, I don't think a change to the default configuration value can be treated as a bug fix. It's more like an enhancement. Because the previous stable releases all should have the same problem. Then, we can see it's not a The Jetty documentation recommends values 50-500 for the maximum thread pool size. That is a fact, so there cannot be different opinions on this. Since the default configuration value doesn't fall in the recommended value range, my opinion is that this is a bug. For some people a bug is a feature. :) Does it really matter whether we call this a bug or an improvement? The fact is that the current value doesn't fall in the recommended value range. -- 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]
