lhotari commented on PR #24359: URL: https://github.com/apache/pulsar/pull/24359#issuecomment-2915738912
This tuning recommendation about increasing `managedLedgerMaxReadsInFlightPermitsAcquireQueueSize` isn't so straight forward since `managedLedgerMaxReadsInFlightSizeInMB` and `managedLedgerMaxReadsInFlightPermitsAcquireTimeoutMillis` settings also impact the situation. If the `managedLedgerMaxReadsInFlightSizeInMB` value is higher, the queuing wouldn't happen in the first place. The general tuning recommendation should be to primarily increase `managedLedgerMaxReadsInFlightSizeInMB`, make more direct memory for the broker and possibly also increase the queue size. When the timeout `managedLedgerMaxReadsInFlightPermitsAcquireTimeoutMillis` is shorter, the queue length would eventually be shorter when reads are rejected due to being in the queue for too long. However, reducing the timeout isn't very helpful since retries would happen eventually and that would just add unnecessary load to the system. -- 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
