rdhabalia commented on pull request #9292: URL: https://github.com/apache/pulsar/pull/9292#issuecomment-768859816
@codelipenghui this improvement serializes long-array to protobuf directly which should not cost cpu, and managed-ledger has a cap on max entries per ledger so, that will make sure to have a limit on size of long array. encoded 150 long shouldn't be a problem in terms of size and cpu as broker persist into bookie into a separate non-io background thread. we can also control size with `managedLedgerMaxUnackedRangesToPersist` config. > I think we should to add a flag for enabling or disabling this enhancement. Sure, let me add the flag. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
