[GitHub] [iceberg] singhpk234 commented on pull request #7262: AWS: abort S3 input stream on close if not EOS

["singhpk234 (via GitHub)" <gi...@apache.org>]

singhpk234 commented on PR #7262:
URL: https://github.com/apache/iceberg/pull/7262#issuecomment-1493398586

   Thanks @bryanck, this is great find and fix !
   
   one doubt will it cause issue when we switch back the client from apache to url-connection (let's say via table props) considering this was not a issue with url-connection client ? 
   
   Also do we need this in 1.2.1 considering 1.2.0 was still shipped with url-connection as default http client ? The regression commit is still in master [commit](https://github.com/apache/iceberg/commit/053028ceb6d3bc04c223a521600b053235bf344a). 


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org