Github user vegaed commented on the issue:
@jvwing Thanks for all the great suggestions and code.
I moved the try up in order to catch those types of parsing exceptions so
now they will route to failure.
Good catch on the integration tests
Yes I agree unit testing is good. I was modeling off of S3 and SQS and
didn't see any there, but I see now that dynamo and kinesis packages are being
both unit and integration tested.
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket