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

Recreate AEH Processor in the event of an error before retrying the processing operation #3614

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MattCosturos
Copy link

@MattCosturos MattCosturos commented Nov 26, 2024

Recreate AEH Processor in the event of an error before retrying the processing operation

Description

Moved the creation and setup of the Azure Event Hub Processor inside the main loop.
The processor.run only returns a non-recoverable error. Therefore we must re-create new processor and setup the subscription.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #3573

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

Recreate AEH Processor in the event of an error before retrying the processing operation

Signed-off-by: MattCosturos <48531957+MattCosturos@users.noreply.github.com>
@MattCosturos MattCosturos requested review from a team as code owners November 26, 2024 14:28
@yaron2
Copy link
Member

yaron2 commented Nov 26, 2024

See failing build tests

@MattCosturos
Copy link
Author

MattCosturos commented Nov 26, 2024

Updated to log the initial connection error, and then retry after 5 seconds. Same way we handle errors after the initial creation.

…processing events in Azure Event Hubs subscription.

Signed-off-by: Matt Costuros <mcosturos@moog.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Azure Event Hub binding stops working if storage act returns 503
2 participants