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

Update to support new namespace for Storm 1.0.0 #32

Closed
wants to merge 2 commits into from
Closed

Update to support new namespace for Storm 1.0.0 #32

wants to merge 2 commits into from

Conversation

ChristophePretty
Copy link

Hi,

Since release 1.0.0 of Storm, the namespace changed from backtype.storm to org.apache.storm.
This patch update all the imports to support the new namespace.
In addition, update pom.xml to use new releases and fix exception handling due to new zookeeper version. (Thanks to christiangda).

Regards,
Christophe

christiangda added a commit to christiangda/kinesis-storm-spout that referenced this pull request Nov 2, 2016
…\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
@pfifer
Copy link
Contributor

pfifer commented Nov 23, 2016

Please confirm that we can use, modify, copy, and redistribute this contribution.

Thanks.

@christiangda
Copy link

This can be used, modified, copied, and redistributed.

El mié., 23 nov. 2016 a las 19:50, Justin Pfifer ([email protected])
escribió:

Please confirm that we can use, modify, copy, and redistribute this
contribution.

Thanks.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#32 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJG_Mny9rAmFeBPrFKXc4hSJfsdkHelks5rBJkGgaJpZM4IWKEs
.

@PrettySimple PrettySimple closed this by deleting the head repository Aug 19, 2022
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.

4 participants