Repository navigation
Add detail to avro exception message - #1505
Conversation
|
@moradology what is going on with this PR? |
69841b4 to
faa831d
Compare
|
I investigated a few paths one might take to address this issue and ended up deciding the best path at this point is also the simplest. It seems as though we might want to consider parameterizing the names of generic avro codecs - this would give us a greater capacity to intelligently handle certain errors and to provide useful information in the case of failure. For instance, the name here (https://github.com/geotrellis/geotrellis/blob/master/spark/src/main/scala/geotrellis/spark/io/avro/codecs/KeyValueRecordCodec.scala#L13) could be parameterized on the names of the underlying Key and Value codecs Here's an example of the exception thrown after applying this PR: |
faa831d to
e75c336
Compare
This PR adds a little bit of information necessary to properly understand why the Avro decoding process fails in some circumstances and is a response to #1477
The new error looks something like: