lizhanhui commented on a change in pull request #384:
URL: 
https://github.com/apache/rocketmq-client-cpp/pull/384#discussion_r735543746



##########
File path: src/main/cpp/rocketmq/ProducerImpl.cpp
##########
@@ -375,12 +375,12 @@ void ProducerImpl::send0(const MQMessage& message, 
SendCallback* callback, std::
   sendImpl(retry_callback);
 }
 
-bool ProducerImpl::endTransaction0(const std::string& target, const 
std::string& message_id,
+bool ProducerImpl::endTransaction0(const std::string& target, const MQMessage& 
message,

Review comment:
       In case we have a message in place, we would no longer need 
trace_context. 




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