Skip to content

[Edge] Make edge executor db access multi instance save#44716

Merged
jscheffl merged 1 commit intoapache:mainfrom
boschglobal:bugfix/make-edge-executor-multi-instance-save
Dec 6, 2024
Merged

[Edge] Make edge executor db access multi instance save#44716
jscheffl merged 1 commit intoapache:mainfrom
boschglobal:bugfix/make-edge-executor-multi-instance-save

Conversation

@AutomationDev85
Copy link
Copy Markdown
Contributor

Description

We see in our multi scheduler deployment cyclic crashes of the scheduler. This points to the issue that different edge executors trying to work on the same DB items during cleaning up the edge jobs items. Add usage of locks so that the edge executor only works which DB items which are not used by other edge executors.

Details about changes

  • Usage of locks on DB items with are used by the edge executor.
  • Locked items are ignored by the other edge executors.

@boring-cyborg boring-cyborg Bot added area:providers provider:edge Edge Executor / Worker (AIP-69) / edge3 labels Dec 6, 2024
Copy link
Copy Markdown
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Thanks for the improvement

@jscheffl jscheffl merged commit 4be8e4d into apache:main Dec 6, 2024
Ohashiro pushed a commit to Ohashiro/airflow that referenced this pull request Dec 6, 2024
Co-authored-by: Marco Küttelwesch <marco.kuettelwesch@de.bosch.com>
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
Co-authored-by: Marco Küttelwesch <marco.kuettelwesch@de.bosch.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:edge Edge Executor / Worker (AIP-69) / edge3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants