jackdaw icon indicating copy to clipboard operation
jackdaw copied to clipboard

EDN serde: Include data in ex-data on deserialization errors

Open r0man opened this issue 6 years ago • 0 comments

When an exception happens while reading EDN data it would be nice to know what the data is that could not be read. Can we add error handling to the EDN serde and include the string that failed to read in the ex-data of the thrown exception?

Similar to what the Avro serdedoes here: https://github.com/FundingCircle/jackdaw/blob/master/src/jackdaw/serdes/avro.clj#L532

I can work on a PR if you are fine with this proposal.

r0man avatar Jan 10 '19 15:01 r0man