RockteMQ-AI commented on issue #1306:
URL: 
https://github.com/apache/rocketmq-clients/issues/1306#issuecomment-5394982705

   **Issue Evaluation**
   
   Category: `enhancement` | Status: **Evaluated**
   
   **Feasibility:** Feasible
   **Scope:** Java client logging infrastructure
   **Compatibility:** Non-breaking — opt-in SPI, default behavior unchanged.
   
   Well-motivated enhancement. Allowing applications to inject a custom logger 
enables structured JSON logging, integration with application logging 
frameworks (Logback, Log4j2), and centralized log management without file-based 
log collection.
   
   The proposed SPI approach (interface + factory) is clean and follows 
standard Java extensibility patterns.
   
   Key considerations:
   - The SPI should be loaded via `ServiceLoader` for zero-config discovery.
   - Default fallback to the current file-based logging when no custom logger 
is provided.
   - Document the SPI contract clearly (thread safety, lifecycle, log levels).
   
   ---
   *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