Shawyeok commented on issue #24694: URL: https://github.com/apache/pulsar/issues/24694#issuecomment-3289607095
@pdolif Good catch! I prefer the first [solution](https://github.com/pdolif/pulsar/pull/14), it handles the case when memory limit is zero inside the `currentUsagePercent` method. @lhotari Btw, should we align the memoryLimit setting and other related settings in `pulsar-perf` with the default values used in the client? That way, we can ensure consistent performance between `pulsar-perf` benchmarks and real-world tests built around PulsarClient. That’s beyond the scope of the fix for this issue. -- 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]
