tianliuliu commented on PR #4132:
URL: https://github.com/apache/rocketmq/pull/4132#issuecomment-1097650194

   > > Some businesses that use the old MQ client have caught these exceptions. 
When they are deleted in this new client, the business compile will fail, 
resulting in incompatible client versions.
   > 
   > Yes, you are right. I think it is nessary to correct the method signature 
as soon as possible to avoid more technical debt. This pr should be a breaking 
change.
   > 
   > # Describe in Chinese
   > 你是对的。但是我认为仍然有必要修正这个错误以避免更多的技术债务,这对于相关方法算是 重大更新(beaking change)。
   > 
   > 在提交此 PR 的过程中也考虑到了尽可能合理的向前兼容,例如 1处调用 永远不会抛出 
2异常`MQBrokerException`、但鉴于该类实现的接口方法定义抛出了`deletePlainAccessConfig`、此处没有去掉`MQBrokerException`。
 <img alt="image" width="818" 
src="https://user-images.githubusercontent.com/18216266/163094159-adb8174c-0ca1-4d8e-9c68-eeff40a7ee36.png";>
   
   link @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