RapperCL opened a new issue, #7512:
URL: https://github.com/apache/rocketmq/issues/7512

   ### Before Creating the Enhancement Request
   
   - [X] I have confirmed that this should be classified as an enhancement 
rather than a bug/feature.
   
   
   ### Summary
   
   
   1 The basic layer's message sending should not involve redundant operations 
such as message body encoding, and its simplicity should be maintained
   
   path: 
org.apache.rocketmq.client.impl.mqclient.MQClientAPIExt#sendMessageAsync 
   
![image](https://github.com/apache/rocketmq/assets/44110731/581d51e7-0707-4877-8639-1c1e678da008)
   
    
   2 The MessageService layer has provided both single message and batch 
message sending methods, and when building a request in the (org. apache. 
locketmq. proxy. processor. ProducerProcessor #) buildSendMessage RequestHeader 
method, it has already determined whether the request is batch requested, so 
there is no need to make a secondary judgment in the MessageService layer.
     So we should simplify the message sending in the basic layer by providing 
only one message sending method, and at the same time, we only need to 
determine whether it is batch based.
     
   
![image](https://github.com/apache/rocketmq/assets/44110731/f9bbeed1-33fe-429f-bc7f-eaa2829dd128)
   
   
   ### Motivation
   
   Simplify message sending at the basic layer and reduce redundant judgments
   
   ### Describe the Solution You'd Like
   
   Simplify message sending at the basic layer and reduce redundant judgments
   
   ### Describe Alternatives You've Considered
   
   。。。
   
   ### Additional Context
   
   _No response_


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

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

Reply via email to