Kinesis upgrade - Githubissues

guardian / mobile-notifications-content

Lambda that reads from the content api stream and triggers notifications for followable content

2 stars 0 forks source link

Kinesis upgrade #51

Closed lindseydew closed 5 months ago

lindseydew commented 5 months ago

What does this change?

This adds a commit that @fredex42 has made to include a lambda deaggregation method, so that the code will still compile after the aws upgrades. https://docs.aws.amazon.com/streams/latest/dev/kinesis-record-deaggregation.html, https://github.com/awslabs/kinesis-aggregation/tree/master/java/KinesisDeaggregatorV2

How to test

How can we measure success?

Have we considered potential risks?

Images

Accessibility

[ ] Tested with screen reader
[ ] Navigable with keyboard
[ ] Colour contrast passed
[ ] The change doesn't use only colour to convey meaning

lindseydew commented 5 months ago

Closing this because it through a runtime exception error: ArrayStoreException. Investigating this further I believe this is because there the version of the deagg lib is using an older version of https://github.com/aws/aws-lambda-java-libs/blob/main/aws-lambda-java-events/RELEASE.CHANGELOG.md before the change of the namespace of the Kinesis Event. Curiously, it looks like from the docs that there are later versions of the library, but they don't seem to be published the maven.