Removing Flink adapter in favor of the one included with Flink - Githubissues

apache / pulsar-adapters

Apache Pulsar Adapters

https://pulsar.apache.org/

24 stars 31 forks source link

Removing Flink adapter in favor of the one included with Flink #34

Closed dlg99 closed 2 years ago

dlg99 commented 2 years ago

Motivation

Removing Flink adapter in favor of https://github.com/apache/flink/tree/master/flink-connectors/flink-connector-pulsar This one is based on an old version of Flink which brings in dependencies with various CVEs, since that version Flink added pulsar connector in their project.

Modifications

Removed Flink adapter, tests, examples, and dependencies.

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, removed
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? removes the feature
If yes, how is the feature documented? not applicable but should be mentioned in the release notes
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

merlimat commented 2 years ago

We could also update the README to mention the adapters that are now developed somewhere else

dlg99 commented 2 years ago

@merlimat I updated readme