-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(pipes-targets): add Kinesis #30656
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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. |
578997d
to
48ddddf
Compare
48ddddf
to
352bcf8
Compare
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
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.