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


##########
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:
   Mailing list thread:
   https://lists.apache.org/thread/kfm7n6xf9pf7h76k4gx83zv3c2kj6yy1
   
   When we say there's support for downgrading (rollback), it doesn't
   mean users don't need to take any action.
   
   We ensure downgrade compatibility by providing an upgrade guide that
   explains how to configure the system to allow rollbacks without losing
   state information. If we didn't do this, we'd always be stuck in the
   same situation whenever a new LTS version comes out.
   
   Let's start making progress:
   - keep PR 9292 as default
   - implement PR 23759 without changing the PR 9292 default
   - write a proper upgrade guide for Pulsar 4.0 where rollback
   considerations are explained



-- 
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