caigy commented on PR #4834:
URL: https://github.com/apache/rocketmq/pull/4834#issuecomment-1220198721

   > Hi, @caigy I have two small tips:
   > 
   > 1. Simplify the code by removing the logic of sending retry
   > 2. Determine whether to reuse the update topic route info in 
AssignmentManager
   
   @RongtongJin Thanks for your advice. 
   
   - Retry logic has been removed to keep escape logic simple, just let caller 
to handle it.
   - `TopicRouteInfoManager` has been created, which maintains for topic routes 
data used by `EscapeBridge` and `AssignmentManager`.


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