RussellSpitzer commented on code in PR #9691:
URL: https://github.com/apache/iceberg/pull/9691#discussion_r1492700348
##########
core/src/main/java/org/apache/iceberg/avro/AvroIterable.java:
##########
@@ -150,7 +153,7 @@ public D next(D reuse) {
try {
return reader.next(reuse);
} catch (IOException e) {
- throw new RuntimeIOException(e, "Failed to read next record");
+ throw new RuntimeIOException(e, "Failed to read next record at file:
%s", file);
}
Review Comment:
I think we should probably add some checks to the below methods as well, we
can Rethrow the IOException but enrich it with a message about which file broke
--
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-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
