ingdex opened a new pull request, #4653: URL: https://github.com/apache/rocketmq/pull/4653
add message property PROPERTY_BORN_TIMESTAMP and set bornTimeStamp in sendMessageRequestHeader as PROPERTY_BORN_TIMESTAMP ## What is the purpose of the change Currently, org.apache.rocketmq.common.protocol.header.SendMessageRequestHeader.bornTimestamp is set as the timestamp proxy sending message to broker, change it to message's bornTimestamp. In RocketMQ 4.0, org.apache.rocketmq.common.protocol.header.SendMessageRequestHeader.bornTimestamp represent the timestamp message born. However, in rocketmq 5.0 message's bornTimestamp is a message property, which is the timestamp message is constructed, while there isn't bornTimestamp in SendMessageRequest of RocketMQ 5.0. Note: When sending multiple messages with grpc sendMessage, bornTimestamp in requestHeader is set to first message's bornTimestamp, which may not be precise. When bornTimestamp is needed, using the bornTimestamp of message property is suggested. -- 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]
