Skip to content
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

Update version of amazon-kinesis-client to v1.4.0 #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Twistedpear
Copy link

The module ships with version 1.2.0 of the amazon-kinesis-client package which doesn't support the aggregation features available in later versions. This pull request updates the package to 1.4.0 which adds the aggregation support.

This build works for me on Mac OS X 10.10 and EC2 Linux. Versions of the amazon-kinesis-client older than 1.4 don't work on EC2 linux by default as they appear to be compiled with Java 1.8, whereas EC2 Linux ships with 1.7/

I've updated the supporting packages as well, though I'm not 100% sure about which version of the original packages are still required.

Rob Lugton added 2 commits October 22, 2015 09:21
The module ships with version 1.2.0 of the kinesis client
which doesn't support aggregation. I've updated some of the
supporting libraries too, though I'm not 100% sure if the
list is correct.

This configuration works for me on EC2 linux and Mac OS X.
Updating the kinesis client beyond 1.4.0 has issues on
EC2 linux as the default java version is 1.7 and it appears
the client has been compiled against 1.8
@nemo
Copy link

nemo commented Oct 26, 2015

👍

@pkopac
Copy link

pkopac commented Mar 17, 2016

Thanks, @Twistedpear , for the update of versions. AWS should update this stuff more often, since Kinesis is cool service, but without up-to-date & fixed libs it can't be used. I tried upgrading to KCL 1.6.1, but some internal API is not compatible anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants