Skip to content
This repository has been archived by the owner on Apr 13, 2018. It is now read-only.

Various pull request applied and update all dependencies #37

Closed
wants to merge 8 commits into from

Conversation

christiangda
Copy link

@christiangda christiangda commented Nov 2, 2016

List of changest

…\n\- applied PR amazon-archives#23, \n\- applied PR amazon-archives#32, \n\- Register MDC key for TOPOLOGY_NAME, \n\- added Maven Encoding default to properties, \n- added gitignore rules, \n\- updated all dependencies versions in pom.xml
@abshkd
Copy link

abshkd commented Nov 16, 2016

no one is maintaining this project anymore, it seems AWWS abandoned storm. I cant find a single working example for Storm 1.x. Any help is appreciated.

@christiangda
Copy link
Author

christiangda commented Nov 22, 2016

Hi @abshkd , yes as you say!, no one is maintaining this project anymore, but my PR work with the sample describe here:

https://github.com/awslabs/kinesis-storm-spout/blob/master/src/main/samples/SampleTopology.java

@pfifer
Copy link
Contributor

pfifer commented Nov 22, 2016

I'm looking into merging this, but don't have an immediate ETA.

@christiangda
Copy link
Author

Hi pfifer!, thanks for your help, The comunity is in nedded.

I'm refactoring my PR, because I'm having problems with curator version, and I Thought that was PR #12

@abshkd
Copy link

abshkd commented Nov 26, 2016

@christiangda You might want to try this instead https://github.com/apache/storm/tree/master/external/storm-kinesis This is better because it matches the curator used in storm. I have it working with very minor issues at the moment. Will post example on Monday.
Edit: Its the same sources btw . only merged with minor changes. Even though the source is in master or 2.x branch you can successfully use it in 1.x. I can confirm that

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

Successfully merging this pull request may close these issues.

3 participants