mapleFU commented on PR #43861: URL: https://github.com/apache/arrow/pull/43861#issuecomment-2315116013
> I didn’t dial an S3 address in Parquet ut; I only mock an error when io.Reader calls ReadAt. We need to ensure any 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. I see. A lot of module under Parquet-go might need handle `error` gracefully. Maybe some fuzzing script to generate bad file or inject bad io-state would help finding them, sigh... -- 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]
