lordcheng10 commented on a change in pull request #13679:
URL: https://github.com/apache/pulsar/pull/13679#discussion_r825423681
##########
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java
##########
@@ -1678,23 +1678,34 @@
private int managedLedgerMaxAckQuorum = 5;
@FieldContext(
category = CATEGORY_STORAGE_ML,
+ dynamic = true,
doc = "Amount of memory to use for caching data payload in managed
ledger. \n\nThis"
+ " memory is allocated from JVM direct memory and it's shared
across all the topics"
+ " running in the same broker. By default, uses 1/5th of
available direct memory")
private int managedLedgerCacheSizeMB = Math.max(64,
(int) (io.netty.util.internal.PlatformDependent.maxDirectMemory()
/ 5 / (1024 * 1024)));
+
+ @FieldContext(
+ category = CATEGORY_STORAGE_ML,
+ dynamic = true,
+ doc = "Threshold to evicteis triggered"
+ )
+ private double evictionTriggerThresholdPercent = 0.98;
Review comment:
Fixed: have removed. @Jason918 PTAL,thanks!
--
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]
