Skip to content

Refactor _manage_executor_state by refreshing TIs in batch #36502

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.

Sign up for GitHub

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

Jump to bottom
Merged
potiuk merged 2 commits into from
Dec 30, 2023
Merged

Refactor _manage_executor_state by refreshing TIs in batch #36502

potiuk merged 2 commits into from
Dec 30, 2023

Conversation

@hussein-awala
Copy link
Member

@hussein-awala hussein-awala commented Dec 30, 2023

#36418 with MSSQL support.

@hussein-awala hussein-awala added the full tests needed We need to run full set of tests for this PR to merge label Dec 30, 2023
@hussein-awala hussein-awala requested a review from potiuk December 30, 2023 20:55
@boring-cyborg boring-cyborg bot added the area:Scheduler including HA (high availability) scheduler label Dec 30, 2023
@hussein-awala
Copy link
Member Author

hussein-awala commented Dec 30, 2023

I cannot test locally because of the following:

MSSQL is not supported on ARM architecture

@potiuk potiuk added this to the Airflow 2.8.1 milestone Dec 30, 2023
potiuk
potiuk reviewed Dec 30, 2023
View reviewed changes
@potiuk
Copy link
Member

potiuk commented Dec 30, 2023

I cannot test locally because of the following:

But it looks like it works now :)

@potiuk potiuk merged commit 9cf5f6f into apache:main Dec 30, 2023
@ephraimbuddy ephraimbuddy added the type:bug-fix Changelog: Bug Fixes label Jan 10, 2024
ephraimbuddy pushed a commit that referenced this pull request Jan 11, 2024
@hussein-awala @ephraimbuddy
Refactor _manage_executor_state by refreshing TIs in batch (#36502)
81b2331 
Refactor _manage_executor_state by refreshing TIs in batch (#36418)" (#36500)"

Handle Microsoft SQL Server

(cherry picked from commit 9cf5f6f)
@ephraimbuddy ephraimbuddy mentioned this pull request Jan 16, 2024
Closed
68 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@potiuk potiuk potiuk approved these changes

@kaxil kaxil Awaiting requested review from kaxil

@ashb ashb Awaiting requested review from ashb

@XD-DENG XD-DENG Awaiting requested review from XD-DENG

Assignees

No one assigned

Labels

area:Scheduler including HA (high availability) scheduler full tests needed We need to run full set of tests for this PR to merge type:bug-fix Changelog: Bug Fixes

Projects

None yet

Milestone

Airflow 2.8.1

Development

Successfully merging this pull request may close these issues.

3 participants

@hussein-awala @potiuk @ephraimbuddy