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

   ### 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/216219128-48a64096-1777-4dfd-940e-ed42139048f9.png)
   located at:
   
eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/handler/ConfigurationHandler.java
   analysis and explanation:
   This method allocates and uses an auto closeable resource. However, it 
manually closes the resource in a finally block. While this is correct 
management, it doesn't rely on the idiomatic way available to JDK 7 and above, 
allows for possible subtle problems, and complicates the reading of code by 
developers expecting the use of try-with-resources.
   
   
   ### Describe the solution you'd like
   
   Change to use try-with-resources to manage resources.
   
   ### 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