Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-30 Thread via GitHub
potiuk merged PR #36502: URL: https://github.com/apache/airflow/pull/36502 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.a

Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-30 Thread via GitHub
potiuk commented on PR #36502: URL: https://github.com/apache/airflow/pull/36502#issuecomment-1872611495 > I cannot test locally because of the following: But it looks like it works now :) -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-30 Thread via GitHub
potiuk commented on code in PR #36502: URL: https://github.com/apache/airflow/pull/36502#discussion_r1438714482 ## airflow/jobs/backfill_job_runner.py: ## @@ -264,16 +264,46 @@ def _manage_executor_state( :return: An iterable of expanded TaskInstance per MappedTask

Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-30 Thread via GitHub
hussein-awala commented on PR #36502: URL: https://github.com/apache/airflow/pull/36502#issuecomment-1872605298 I cannot test locally because of the following: > MSSQL is not supported on ARM architecture -- This is an automated message from the Apache Git Service. To respond to the mes

Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-30 Thread via GitHub
potiuk commented on PR #36418: URL: https://github.com/apache/airflow/pull/36418#issuecomment-1872525803 It fails in main for MSSQL. We are planning to cherry-pick that one for 2.8.1 so I hope it's the last that requires MSSQL special treatment (I guess you need to open fixup PR and set `f

Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-30 Thread via GitHub
potiuk merged PR #36418: URL: https://github.com/apache/airflow/pull/36418 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.a

Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-29 Thread via GitHub
potiuk commented on PR #36418: URL: https://github.com/apache/airflow/pull/36418#issuecomment-1871846529 Nice one! @hussein-awala -> I just wonder if that could be the reason for https://github.com/apache/airflow/discussions/36454 -- This is an automated message from the Apache Git Servi

Re: [PR] Refactor _manage_executor_state by refreshing TIs in batch [airflow]

2023-12-25 Thread via GitHub
hussein-awala closed pull request #36418: Refactor _manage_executor_state by refreshing TIs in batch URL: https://github.com/apache/airflow/pull/36418 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to