-
Notifications
You must be signed in to change notification settings - Fork 14
Potentially switch to implementing IRecordProcessor #6
Comments
Hi Brandon - yep I think we should switch to using Perhaps we should suggest cloudify puts a warning on his direct read API methods? BTW, for some examples of AWS using |
Hi @alexanderdean, On the IRecordProcessor, I'm starting by getting the java example working, and then I'll convert everything to Scala and tweak to what we need. Though, one problem is that, per the documentation, using the IRecordProcessor requires DynamoDB access:
I don't think this can be circumvented because IRecordProcessor is passed into the amazon-kinesis-client library. The main entry point is in the Worker class, which doesn't seem to provide an alternate to using DynamoDB. Do you still want to switch to implementing IRecordProcessor? If so, can you give me minimal DynamoDB permissions? Running the sample code gives the following error:
|
Hi @bamos - yes I think DynamoDB is pretty tightly coupled into Kinesis to manage state (stream position etc), so can't be avoided really! I will give you DynamoDB permissions and update this ticket when you have them... |
Hi Brandon, you should be good to go now. Let me know if there are any issues with your access. |
Thanks, works great! |
Hi @alexanderdean, I've merged everything into the |
Nice work! |
Hi @alexanderdean, the Amazon documentation for developing a consumer application recommends implementing an IRecordProcessor instead of using the AWS library directly. Do you think we should switch to using it?
From the documentation:
The text was updated successfully, but these errors were encountered: