DanielLeens commented on issue #10531:
URL: https://github.com/apache/seatunnel/issues/10531#issuecomment-4849339869

   I checked the current `GZ` branch in `AbstractReadStrategy`, and this is 
still a real resource-management bug.
   
   The issue is not just stylistic consistency with the other compression 
branches. In the `GZ` path, `GzipCompressorInputStream` is created without 
try-with-resources, so if `readProcess(...)` fails, both the gzip stream and 
the underlying Hadoop input stream can remain open.
   
   There is active follow-up for this now in PR #11209, and PR #10532 also 
includes the same `GZ` cleanup as part of a broader resource-leak patch.
   
   So this issue should stay open until one of those fixes is merged into 
`dev`. There is not really a config-level workaround here besides carrying a 
local patch in the meantime.


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