You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by rc...@apache.org on 2022/09/19 02:58:28 UTC

[james-project] 03/07: JAMES-3816 Allow turning off ReactiveThrottler

This is an automated email from the ASF dual-hosted git repository.

rcordier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git

commit 72c5ebbb3be1cc5e7915102b2727c9d9a238089c
Author: Benoit Tellier <bt...@linagora.com>
AuthorDate: Fri Sep 9 15:37:53 2022 +0700

    JAMES-3816 Allow turning off ReactiveThrottler
---
 .../src/main/java/org/apache/james/imap/api/ImapConfiguration.java     | 1 -
 .../main/java/org/apache/james/imapserver/netty/ReactiveThrottler.java | 3 +++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/protocols/imap/src/main/java/org/apache/james/imap/api/ImapConfiguration.java b/protocols/imap/src/main/java/org/apache/james/imap/api/ImapConfiguration.java
index 21f8928774..5fbbfb4893 100644
--- a/protocols/imap/src/main/java/org/apache/james/imap/api/ImapConfiguration.java
+++ b/protocols/imap/src/main/java/org/apache/james/imap/api/ImapConfiguration.java
@@ -75,7 +75,6 @@ public class ImapConfiguration {
         }
 
         public Builder concurrentRequests(int concurrentRequests) {
-            Preconditions.checkArgument(concurrentRequests > 0, "concurrentRequests should not be zero or negative");
             this.concurrentRequests = Optional.of(concurrentRequests);
             return this;
         }
diff --git a/server/protocols/protocols-imap4/src/main/java/org/apache/james/imapserver/netty/ReactiveThrottler.java b/server/protocols/protocols-imap4/src/main/java/org/apache/james/imapserver/netty/ReactiveThrottler.java
index 67df59b84f..0d9028d556 100644
--- a/server/protocols/protocols-imap4/src/main/java/org/apache/james/imapserver/netty/ReactiveThrottler.java
+++ b/server/protocols/protocols-imap4/src/main/java/org/apache/james/imapserver/netty/ReactiveThrottler.java
@@ -50,6 +50,9 @@ public class ReactiveThrottler {
     }
 
     public Mono<Void> throttle(Publisher<Void> task) {
+        if (maxConcurrentRequests < 0) {
+            return Mono.from(task);
+        }
         int requestNumber = concurrentRequests.incrementAndGet();
 
         if (requestNumber <= maxConcurrentRequests) {


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org