RockteMQ-AI commented on issue #11080:
URL: https://github.com/apache/rocketmq/issues/11080#issuecomment-5584832329

   **Issue Evaluation**
   
   Category:  | Status: **Evaluated**
   
   **Feasibility:** Feasible
   **Scope:** ,  startup path
   **Compatibility:** No breaking changes — clients with a listener continue to 
work identically
   
   This is a valid optimization. The analysis is correct:  unconditionally 
starts  even when  is , resulting in an idle thread polling an empty queue 
every 3 seconds. In Proxy deployments with multiple  instances, this cost 
accumulates.
   
   The proposed fix — guarding  with a  check — is minimal and safe.  already 
handles the never-started case gracefully.
   
   **Suggested implementation notes:**
   - The guard should be placed in  before the  call
   - Verify that  after  still works correctly (if the listener is set 
dynamically post-startup)
   - Add a unit test confirming no  thread appears when no listener is 
configured
   
   ---
   *Automated evaluation by github-manager*


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