-
Notifications
You must be signed in to change notification settings - Fork 222
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
Updated dependency versions #137
Conversation
README.md
Outdated
* Updated to version 2.3.4 of the Amazon Kinesis Client Library for Java. | ||
* Upgraded multiple dependencies |
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.
We should make release notes / dependency upgrades separate PRS, and we should link PRS once they're merged in so customers can reference them in release notes. As part of the release PR, we also need to update versioning for this package's release.
Also, we need to get these release notes approved by doc writer.
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.
Also good to list what dependencies were upgraded specifically (i.e AWS SDK, Jackson, etc.)
How did we test this? |
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.
Left a few comments.
Tested by running the sample https://github.com/awslabs/amazon-kinesis-client-python#running-the-sample. Was able to successfully write and read records from Kinesis Stream. |
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.
LGTM
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.