----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5197/ -----------------------------------------------------------
Review request for pig. Summary ------- The AvroStorage loadfunc throws an IOException when it tries to load a file that is not a valid avro container file. However, it does not currently specify the name of the bad file. This can make debugging a failed map-reduce job frustrating when the number of input files is large. This patch adds the input filename to the IOException. This addresses bug PIG-2709. https://issues.apache.org/jira/browse/PIG-2709 Diffs ----- /trunk/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/PigAvroRecordReader.java 1341337 Diff: https://reviews.apache.org/r/5197/diff Testing ------- Manual verification. Thanks, Mike