Roiocam commented on issue #71:
URL: 
https://github.com/apache/rocketmq-eventbridge/issues/71#issuecomment-1702195279

   Do not post too much code in an issue; it is hard to read.
   
   For example, in cases like ThreadPoolTaskExecutor, consider extending 
ThreadPoolExecutor instead of delegating it. Some getter and setter methods 
have meaningless comments.
   
   There are methods to achieve back pressure, depending on how the producer 
delivers records to the consumer:
   
   When the producer communicates with the consumer through a queue, you can 
use a fixed queue size to achieve this. When the producer cannot push to the 
queue, it immediately goes into idle mode. There should be a common 
specification for this everywhere.
   
   If the producer communicates with the consumer through some RPC mechanism, 
you can let the consumer poll from the producer's delivery buffer. This works 
well in a P2P mode, but once you have multiple consumers for one producer, you 
must fix the delivery buffer size and react to it, which means applying back 
pressure to the producer upstream.
   
   


-- 
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]

Reply via email to