[GitHub] [airflow] Taragolis commented on a diff in pull request #33720: Optimise and migrate to SA2-compatible syntax for TaskReschedule

2023-08-31 Thread via GitHub
Taragolis commented on code in PR #33720: URL: https://github.com/apache/airflow/pull/33720#discussion_r1311284425 ## airflow/models/taskreschedule.py: ## @@ -142,9 +154,93 @@ def find_for_task_instance( :param try_number: Look for TaskReschedule of the given

[GitHub] [airflow] Taragolis commented on a diff in pull request #33720: Optimise and migrate to SA2-compatible syntax for TaskReschedule

2023-08-31 Thread via GitHub
Taragolis commented on code in PR #33720: URL: https://github.com/apache/airflow/pull/33720#discussion_r1311257524 ## airflow/models/taskreschedule.py: ## @@ -142,9 +154,93 @@ def find_for_task_instance( :param try_number: Look for TaskReschedule of the given

[GitHub] [airflow] Taragolis commented on a diff in pull request #33720: Optimise and migrate to SA2-compatible syntax for TaskReschedule

2023-08-30 Thread via GitHub
Taragolis commented on code in PR #33720: URL: https://github.com/apache/airflow/pull/33720#discussion_r1310041422 ## airflow/models/taskreschedule.py: ## @@ -142,9 +154,93 @@ def find_for_task_instance( :param try_number: Look for TaskReschedule of the given

[GitHub] [airflow] Taragolis commented on a diff in pull request #33720: Optimise and migrate to SA2-compatible syntax for TaskReschedule

2023-08-30 Thread via GitHub
Taragolis commented on code in PR #33720: URL: https://github.com/apache/airflow/pull/33720#discussion_r1310007883 ## airflow/models/taskreschedule.py: ## @@ -142,9 +154,93 @@ def find_for_task_instance( :param try_number: Look for TaskReschedule of the given

[GitHub] [airflow] Taragolis commented on a diff in pull request #33720: Optimise and migrate to SA2-compatible syntax for TaskReschedule

2023-08-29 Thread via GitHub
Taragolis commented on code in PR #33720: URL: https://github.com/apache/airflow/pull/33720#discussion_r1309348422 ## airflow/models/taskreschedule.py: ## @@ -142,9 +154,93 @@ def find_for_task_instance( :param try_number: Look for TaskReschedule of the given

[GitHub] [airflow] Taragolis commented on a diff in pull request #33720: Optimise and migrate to SA2-compatible syntax for TaskReschedule

2023-08-29 Thread via GitHub
Taragolis commented on code in PR #33720: URL: https://github.com/apache/airflow/pull/33720#discussion_r1306689074 ## tests/ti_deps/deps/test_ready_to_reschedule_dep.py: ## @@ -64,118 +76,127 @@ def _get_mapped_task_reschedule(self, reschedule_date): ) return

[GitHub] [airflow] Taragolis commented on a diff in pull request #33720: Optimise and migrate to SA2-compatible syntax for TaskReschedule

2023-08-25 Thread via GitHub
Taragolis commented on code in PR #33720: URL: https://github.com/apache/airflow/pull/33720#discussion_r1305460117 ## airflow/models/taskreschedule.py: ## @@ -94,18 +96,16 @@ def __init__( self.reschedule_date = reschedule_date self.duration = (self.end_date -