Alonexc opened a new issue, #3416:
URL: https://github.com/apache/incubator-eventmesh/issues/3416

   ### Search before asking
   
   - [X] I had searched in the 
[issues](https://github.com/apache/eventmesh/issues?q=is%3Aissue) and found no 
similar issues.
   
   
   ### Enhancement Request
   
   
![image](https://user-images.githubusercontent.com/91315508/224205494-ed03a955-28d1-486e-bbe9-3a2a6edda594.png)
   located at:
   
eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/http/consumer/HttpClientGroupMapping.java
 line 391,449
   
   analysis and explanation:
   Because Double Brace Initialization (DBI) creates an anonymous class with a 
reference to the instance of the owning object, its use can lead to memory 
leaks if the anonymous inner class is returned and held by other objects. Even 
when there’s no leak, DBI is so obscure that it’s bound to confuse most 
maintainers.
   For collections, use Arrays.asList instead, or explicitly add each item 
directly to the collection.
   
   
   ### Describe the solution you'd like
   
   
![image](https://user-images.githubusercontent.com/91315508/224205525-731f408f-4e55-4ddc-b92a-2315a839e85a.png)
   
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to