Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[receiver/azureeventhub] Azure eventhub impl #12795

Closed
wants to merge 1 commit into from

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Aug 1, 2022

Description:
Implementation of Azure event hub receiver component.

Link to tracking Issue:
#12786

Testing:
Comprehensive unit test testing. The hub itself is not tested, still WIP.

Documentation:
README and testdata folder.

@atoulme
Copy link
Contributor Author

atoulme commented Aug 1, 2022

@djaglowski FYI this impl (it's WIP) is using the storage experimental extension to keep track of event hub checkpoints. I thought you might like to review at leisure.

@atoulme atoulme force-pushed the azure_eventhub_impl branch 2 times, most recently from 4846c38 to c4f0a36 Compare August 1, 2022 14:36
@atoulme atoulme changed the title Azure eventhub impl [receiver/azureeventhub] Azure eventhub impl Aug 1, 2022
@djaglowski
Copy link
Member

@atoulme, the usage of the storage extension looks good to me.

You got me thinking about #10915, which I think will eventually require some small changes on this component. I worked up a potential solution in #12871. If that PR is merged first, I can help roll out the associated changes here.

@atoulme atoulme force-pushed the azure_eventhub_impl branch from c4f0a36 to 1ca5247 Compare August 13, 2022 22:57
@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Aug 28, 2022
@github-actions
Copy link
Contributor

Closed as inactive. Feel free to reopen if this PR is still being worked on.

@github-actions github-actions bot closed this Sep 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants