[ 
https://issues.apache.org/jira/browse/ARTEMIS-4453?focusedWorklogId=884175&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-884175
 ]

ASF GitHub Bot logged work on ARTEMIS-4453:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 09/Oct/23 19:58
            Start Date: 09/Oct/23 19:58
    Worklog Time Spent: 10m 
      Work Description: clebertsuconic commented on code in PR #4643:
URL: https://github.com/apache/activemq-artemis/pull/4643#discussion_r1350742406


##########
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientProducerCreditManagerImpl.java:
##########
@@ -16,20 +16,29 @@
  */
 package org.apache.activemq.artemis.core.client.impl;
 
+import java.lang.invoke.MethodHandles;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.Map;
+import java.util.concurrent.Executor;
 import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.ScheduledFuture;
 import java.util.concurrent.TimeUnit;
 
 import org.apache.activemq.artemis.api.core.SimpleString;
+import org.apache.activemq.artemis.core.server.ActiveMQScheduledComponent;
 import org.apache.activemq.artemis.spi.core.remoting.SessionContext;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class ClientProducerCreditManagerImpl implements 
ClientProducerCreditManager {
 
+   private static final Logger logger = 
LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
+
    public static final int MAX_UNREFERENCED_CREDITS_CACHE_SIZE = 1000;
 
+   private static final int MAX_UNREFERENCED_CREDITS_CACHE_TIME_CHECK = 500; 
// milliseconds

Review Comment:
   I set this as half second just for the test... but I'm actually thinking 
this should be something with a short timeout like this.. as long as the 
cleanup works.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 884175)
    Time Spent: 1h 50m  (was: 1h 40m)

> Bridge blocked by flow control, seemingly forever
> -------------------------------------------------
>
>                 Key: ARTEMIS-4453
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-4453
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>          Components: Broker, Clustering
>            Reporter: Andy Gustafson
>            Assignee: Justin Bertram
>            Priority: Major
>          Time Spent: 1h 50m
>  Remaining Estimate: 0h
>
> I found what I believe to be a bug related to flow control with symmetric 
> clusters. I wrote [this 
> example|https://github.com/acgustafson/artemis-flowcontrol-issue] to 
> reproduce  the behavior I am seeing. The test program does the following:
>  * Create 2 clustered embedded brokers
>  * Create 3000 topics with 1 producer per topic sending 1 message every 30 
> seconds with 1500 producers on each broker.
>  * Create 12 consumers on a wildcard that matches all 3000 topics with 6 
> consumers on each broker
>  * Create 1 additional "statistics" wildcard consumer to log how many 
> messages it has consumed over the previous 30 seconds
> Shortly after running this program the "statistics consumer" shows it's only 
> consuming 1500 messages per 30 seconds. The broker's logs show the 
> $artemis.internal.sf queues are being blocked on flow control. From what I 
> can tell the broker never seems to recover from this, even after killing all 
> of the producers/consumers and restarting with a much lower number of 
> producers/consumers. 
> This behaviour started appearing for me in 2.22.0 after 
> https://issues.apache.org/jira/browse/ARTEMIS-3805 changed the default 
> producer-window-size to 1MB from -1.
>  
> I can reproduce the issue using the latest snapshot. If you set the 
> producer-window-size back to -1 the issue no longer appears. In my test 
> program I used wildcards, but I was able to reproduce this without wildcards 
> as well.
>  
> Program to reproduce: 
> [https://github.com/acgustafson/artemis-flowcontrol-issue]
>  
> Discussion of this issue in slack at 
> [https://the-asf.slack.com/archives/CFL910J30/p1696356372797149?thread_ts=1694705189.957139&cid=CFL910J30]



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to