You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2022/10/16 04:51:34 UTC

[GitHub] [rocketmq] nowinkeyy commented on issue #5307: Whether the if statement is redundant?

nowinkeyy commented on issue #5307:
URL: https://github.com/apache/rocketmq/issues/5307#issuecomment-1279891645

   Hi @complone , thank you for your reply.
   I know what you mean. IMO, it is sufficient to run ```getConfirmOffset()``` directly because it includes ```isDuplicationEnable()``` without running ```isDuplicationEnable()``` before running ```getConfirmOffset()```.


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

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