Delete Storm adapter (it is moved to Apache Storm community) - Githubissues

apache / pulsar-adapters

Apache Pulsar Adapters

https://pulsar.apache.org/

24 stars 31 forks source link

Delete Storm adapter (it is moved to Apache Storm community) #47

Closed dlg99 closed 1 year ago

dlg99 commented 1 year ago

Motivation

As discussed/decided during a community meeting, Storm Adapter is passed to the Apache Storm community. Since it was accepted there we can delete the code in this repo.

See https://github.com/apache/storm/tree/master/external/storm-pulsar

Modifications

Deleting Storm-related code.

Verifying this change

[ ] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

tisonkun commented 1 year ago

This state looks good to me for merging. Is there any other concern or blocker?