bigsheeper commented on PR #43861: URL: https://github.com/apache/arrow/pull/43861#issuecomment-2315108147
> Dial a s3 address in parquet ut would be a disaster and be unstable, I don't suggest that > > Mocking might be ok here I didn’t dial an S3 address in Parquet ut; I only mock an error when `io.Reader` calls `ReadAt`. The connection timeout to S3 is a real issue I encountered in system. We need to ensure this error is thrown and handled properly, otherwise users might face situations where they read 0 rows of Parquet data, which would be disastrous for them. -- 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]
