lhotari opened a new pull request, #15748:
URL: https://github.com/apache/pulsar/pull/15748
### Motivation
- There's a regression with the tools since the memory limit cannot be
adjusted
- It's better to default to the previous setting of disabling memory limits
so that the performance profile doesn't change because of the memory
limit.
- Memory limit is enabled by default in "PIP-120: Enable client memory limit
by default", #13306 . PR was #13344 .
### Modifications
Set `.memoryLimit(0, SizeUnit.BYTES)` for PulsarClient builders in Cli tools.
### Additional context
#15723 restores the default `maxPendingMessages` &
`maxPendingMessagesAcrossPartitions` limits for producers to the defaults used
before PIP-120 changes when the memory limit is disabled. That might be
necessary in some cases for restoring the pre-PIP-120 behavior.
--
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]
