feat(pipes-targets): add Kinesis#30656
Conversation
nmussy
left a comment
There was a problem hiding this comment.
Just a few nits, should be good to go otherwise
fed3b6f to
d0df247
Compare
|
@nmussy Updated as well. Thanks again! |
|
This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state. |
a9082c3 to
e0f8e5f
Compare
e0f8e5f to
6ab7aa1
Compare
|
This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state. |
48ddddf to
352bcf8
Compare
Leo10Gama
left a comment
There was a problem hiding this comment.
Thanks for the PR! Just a few minor changes/suggestions from my end.
352bcf8 to
b1df332
Compare
Pull request has been modified.
|
@Leo10Gama – fixed. Thanks for the review! |
Leo10Gama
left a comment
There was a problem hiding this comment.
LGTM, thanks for the contribution!
|
@Mergifyio update |
✅ Branch has been successfully updated |
|
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
|
Comments on closed issues and PRs are hard for our team to see. |
Add Kinesis data stream as a Pipes target.
It's nontrivial to get the data from the Kinesis data stream, but here are screenshots
showing data made it through during the integration test.