luozongle01 opened a new pull request, #8807:
URL: https://github.com/apache/rocketmq/pull/8807

   <!-- Please make sure the target branch is right. In most case, the target 
branch should be `develop`. -->
   
   ### Which Issue(s) This PR Fixes
   
   <!-- Please ensure that the related issue has already been created, and 
[link this pull request to that issue using 
keywords](<https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword>)
 to ensure automatic closure. -->
   
   Fixes #8806
   
   ### Brief Description
   1. Delay the creation of the ProducerAccumulator until startup. Since 
instanceName will change at startup, and unitName should already be set at this 
time.
   2. Due to delaying the Producer Accumulator to start, in order to allow 
users to set the parameters of batchMaxDelayMs, batchMaxbytes, and totaled 
BatchMaxbytes in the Producer Accumulator before executing the start method 
(Because I am worried that some users may set these parameters before executing 
start), three properties have been added to the DefaultMQProducer. After the 
ProducerAccumulator is initialized, it is assigned to the ProducerAccumulator.
   
   ### How Did You Test This Change?
   
   <!-- In order to ensure the code quality of Apache RocketMQ, we expect every 
pull request to have undergone thorough testing. -->
   I have tested locally and confirmed that it will not be sent to other 
RocketMQ clusters, and the parameter settings can take effect.
   <img width="1246" alt="image" 
src="https://github.com/user-attachments/assets/6443b84c-ab87-4b57-8adf-01af77272ea3";>
   <img width="1284" alt="image" 
src="https://github.com/user-attachments/assets/76082d49-3ded-4be8-bb57-6400a544aabb";>
   
   


-- 
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...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to