rtrivedi12 opened a new pull request, #6010: URL: https://github.com/apache/hive/pull/6010
…tially written proto files ### What changes were proposed in this pull request? This PR introduces a more lenient version of ProtobufInputFormat that ignores EOFException when reading partially written proto files. ProtobufMessageInputFormat skips zero-byte files by creating <SafeRecordReader>, which ignores EOFException. ### Why are the changes needed? Abrupt AM termination or OOM failures can result in partially written protobuf files. When reading data from the corresponding external table, these incomplete files may trigger an EOFException, causing the entire query to fail. To improve resilience, it would be better to gracefully skip unreadable or corrupted proto files and continue reading the remaining valid data, instead of failing the entire query. ### Does this PR introduce _any_ user-facing change? Yes, this would silently skip reading partially written proto files instead of failing with EOFException. ### How was this patch tested? Manually tested with a sample partially written dag data file. -- 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]
