kinolaev commented on PR #15792: URL: https://github.com/apache/iceberg/pull/15792#issuecomment-4680917505
The PR is not stale. It is not critical for Parquet in version 1.11.0 because, after PR #13997, the Parquet reader calls S3InputStream.readFully instead of S3InputStream.openStream. However, it is still useful for 1.10.x, where the rewrite_data_files procedure often gets stuck on files with several row groups. It might also be useful for formats other than Parquet if they call S3InputStream.openStream and pause reading for longer than a server timeout. All comments have been addressed. The PR is now waiting for a committer. -- 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]
