Skip to content
This repository has been archived by the owner on Mar 30, 2021. It is now read-only.

ITransformer: Using java.lang.Exception instead of IOException #46

Open
darkcrawler01 opened this issue Apr 20, 2015 · 1 comment
Open

Comments

@darkcrawler01
Copy link

Hi

I feel that defining toClass & fromClass to throw IOException is very constraining.
In case, I am performing some validation and want to throw some custom Exception I cannot do that.
Also the KinesisConnectorRecordProcessor only handles IOExceptions, its even handle any other Runtime Exceptions that may occur. This is not a valid reason to stop processing the rest of the records.
Hence I request you to change the use java.lang.Exception or at the very least use RuntimeExceptions instead everywhere

@workanandr
Copy link

I second this request. Recently, in my application, the worker went into an infinite loop due to this issue. A non-IOException was thrown due to a data processing error and because of the above issue, the RecordProcessor never checkpointed the record. So it kept processing the same record over and over again. I had to catch all exceptions in my code and throw an "IOException" to get past this issue. I used kinesis-connectors version 1.1.2.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants