Oliverwqcwrw commented on PR #5076:
URL: https://github.com/apache/rocketmq/pull/5076#issuecomment-1247631952

   > IMO, This method is invoked by users, and the fellow property is used to 
check whether the message is a timer message in HookUtils.
   > 
   > ```java
   > public static final String PROPERTY_TIMER_DELAY_MS = "TIMER_DELAY_MS";
   > ```
   Hello @ni-ze ,
   I think these two parameters (PROPERTY_TIMER_DELAY_MS and 
PROPERTY_TIMER_DELIVER_MS)express the same effect, whether we can remove the 
redundant parameters related to the logic. Please help review  @RongtongJin 


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