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