thetumbled commented on code in PR #23759:
URL: https://github.com/apache/pulsar/pull/23759#discussion_r1914559497


##########
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java:
##########
@@ -2243,6 +2243,10 @@ The max allowed delay for delayed delivery (in 
milliseconds). If the broker rece
             + " will only be tracked in memory and messages will be 
redelivered in case of"
             + " crashes.")
     private int managedLedgerMaxUnackedRangesToPersist = 10000;
+    @FieldContext(
+            category = CATEGORY_STORAGE_ML,
+            doc = "Whether persist cursor ack stats as long arrays, which will 
compress the data and reduce GC rate")
+    private boolean managedLedgerPersistIndividualAckAsLongArray = false;

Review Comment:
   I support that keeping compatibility should comes first, which impact most 
of the users.
   and i wonder whether we can fix the issue without introducing the switch to 
disable the improvement introduced by #9292. 



-- 
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: commits-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to