Github user vegaed commented on the issue: https://github.com/apache/nifi/pull/1125 @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 with INFRA. ---