You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rocketmq.apache.org by "miles-ton (via GitHub)" <gi...@apache.org> on 2024/05/20 16:23:12 UTC

[I] [Enhancement] Reduce try catch scope in ProcessQueue#removeMessage [rocketmq]

miles-ton opened a new issue, #8171:
URL: https://github.com/apache/rocketmq/issues/8171

   ### Before Creating the Enhancement Request
   
   - [X] I have confirmed that this should be classified as an enhancement rather than a bug/feature.
   
   
   ### Summary
   
   the try catch only needs to catch the lockInterruptibly
   ![image](https://github.com/apache/rocketmq/assets/77013030/0da71730-54ba-414a-9b0e-912cc8510e24)
   
   
   ### Motivation
   
   clean code
   
   ### Describe the Solution You'd Like
   
   just catching up on lockInterruptibly
   
   ### Describe Alternatives You've Considered
   
   none
   
   ### Additional Context
   
   _No response_


-- 
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: commits-unsubscribe@rocketmq.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] [Enhancement] Reduce try catch scope in ProcessQueue#removeMessage [rocketmq]

Posted by "miles-ton (via GitHub)" <gi...@apache.org>.
miles-ton closed issue #8171: [Enhancement] Reduce try catch scope in ProcessQueue#removeMessage
URL: https://github.com/apache/rocketmq/issues/8171


-- 
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: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org