Add detail to avro exception message #1505
Conversation
moradology
changed the title
|
@moradology what is going on with this PR? |
moradology
force-pushed
the
fix/avro-type-error-detail
branch
from
69841b4 to
faa831d
Compare
moradology
changed the title
|
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: |
moradology
force-pushed
the
fix/avro-type-error-detail
branch
from
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: