lhotari commented on code in PR #23340:
URL: https://github.com/apache/pulsar/pull/23340#discussion_r1771809739
##########
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java:
##########
@@ -1231,14 +1231,14 @@ The max allowed delay for delayed delivery (in
milliseconds). If the broker rece
doc = "On Shared and KeyShared subscriptions, if all available
messages in the subscription are filtered "
+ "out and not dispatched to any consumer, message
dispatching will be rescheduled with a backoff "
+ "delay. This parameter sets the initial backoff delay in
milliseconds.")
- private int dispatcherRetryBackoffInitialTimeInMs = 100;
+ private int dispatcherRetryBackoffInitialTimeInMs = 0;
Review Comment:
We shouldn't disable the limit. Setting it to a very low value like 10 ms or
even 1 ms is better for saving CPU and preventing a spinning loop that burns a
lot of CPU.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
