acvictor opened a new pull request, #11617:
URL: https://github.com/apache/incubator-gluten/pull/11617

   ## What changes are proposed in this pull request?
   This PR refactors `ResourceUtil.getResourcesFromJarFile()` to use 
try-with-resources for `ZipFile`, replacing the manual open/close pattern. The 
existing code opens a file in one try block and closes it in a separate try 
block at the end of the method. If any unchecked exception is thrown while 
iterating over the zip entries, `ZipFile.close()` is never called, leaking a 
file descriptor. 
   
   ## How was this patch tested?
   Existing unit tests.
   
   ## Was this patch authored or co-authored using generative AI tooling?
   No.
   


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