RockteMQ-AI commented on issue #10972:
URL: https://github.com/apache/rocketmq/issues/10972#issuecomment-5440279754

   **Issue Evaluation**
   
   Category: bug | Status: **Confirmed**
   
   This is a valid bug. The encoding order in 
`TimerMessageStore#convertMessage` is incorrect: propertiesString is encoded 
before REAL_TOPIC/REAL_QUEUE_ID are cleared from the map, causing the delivered 
message to carry stale routing metadata that diverges from the RocksDB timer 
store path.
   
   **Root Cause:** Encode-then-clear ordering — the properties map is 
serialized to string before the clear operations, so the encoded string still 
contains the original REAL_TOPIC/REAL_QUEUE_ID values.
   **Impact:** Timer-delivered messages may have incorrect routing metadata, 
potentially causing message misdelivery or duplicate processing.
   **Severity:** Medium — affects timer message correctness.
   
   A fix should reverse the order: clear REAL_TOPIC/REAL_QUEUE_ID first, then 
encode propertiesString.
   
   ---
   *Automated evaluation by github-manager*


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