poorbarcode commented on PR #24423: URL: https://github.com/apache/pulsar/pull/24423#issuecomment-2982729421
@lhotari > The problem with this PR is that the limit is per connection to the broker. That's not very effective in limiting the broker memory usage. In Netty, the standard way to handle this is to change autoread to false when writability changes to false. However, that could cause performance issues for Pulsar broker. Changed the implementation to setting channel auto-read to `fasle`, but the drawback is that it also affects connection health check. -- 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]
