Skip to content

Conversation

@alpinegizmo
Copy link
Contributor

  • General
    • The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
    • The pull request addresses only one issue
    • Each commit in the PR has a meaningful commit message (including the JIRA id)

Added a section to the connectors overview page that lists the connectors in Bahir, with links to the docs for each.


The [Apache Bahir](http://bahir.apache.org/) project provides some additional streaming connectors for Flink, including:

* [Apache ActiveMQ](http://bahir.apache.org/docs/flink/current/flink-streaming-activemq/) (sink/source)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe switch sink/source?

The [Apache Bahir](http://bahir.apache.org/) project provides some additional streaming connectors for Flink, including:

* [Apache ActiveMQ](http://bahir.apache.org/docs/flink/current/flink-streaming-activemq/) (sink/source)
* [Apache Flume](http://bahir.apache.org/docs/flink/current/flink-streaming-flume/) (sink)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use https instead?


## Connectors in Apache Bahir

The [Apache Bahir](https://bahir.apache.org/) project provides some additional streaming connectors for Flink, including:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove "some". Consider reversing and expanding the statement, something like "Additional streaming connectors for Flink are being released through Apache Bahir, including:". This could help acclimate contributors and treats Bahir less like a third-party.

@greghogan
Copy link
Contributor

@alpinegizmo thanks for the contribution! Merging ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants