RockteMQ-AI commented on issue #10962: URL: https://github.com/apache/rocketmq/issues/10962#issuecomment-5397176217
**Issue Evaluation** Category: | Status: **Evaluated** **Feasibility:** Feasible **Scope:** module — **Compatibility:** No breaking changes The current implementation validates against for **all** actions (line 119–121 in ), including remove actions. This is logically inconsistent — quota validation is designed to limit subscription **growth**, but remove actions (, ) **reduce** the active subscription set and should not be blocked. **Proposed fix:** Wrap the call inside a condition that checks for add actions only: This is a targeted, low-risk change. The quota protection for subscription growth is preserved while allowing large unsubscribe requests to proceed normally. --- *Automated evaluation by RockteMQ-AI* -- 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]
