Skip to content

Remove AIP-44 from models/taskinstance#44510

Closed
rawwar wants to merge 9 commits intoapache:mainfrom
rawwar:kalyan/AIP-44/remove/models/ti
Closed

Remove AIP-44 from models/taskinstance#44510
rawwar wants to merge 9 commits intoapache:mainfrom
rawwar:kalyan/AIP-44/remove/models/ti

Conversation

@rawwar
Copy link
Copy Markdown
Contributor

@rawwar rawwar commented Nov 30, 2024

related to #44436

  • ./airflow/models/taskinstance.py: _merge_ti
  • ./airflow/models/taskinstance.py: _add_log
  • ./airflow/models/taskinstance.py: _update_ti_heartbeat
  • ./airflow/models/taskinstance.py: _xcom_pull
  • ./airflow/models/taskinstance.py: _get_template_context
  • ./airflow/models/taskinstance.py: _handle_failure
  • ./airflow/models/taskinstance.py: _record_task_map_for_downstreams
  • ./airflow/models/taskinstance.py: _update_rtif
  • ./airflow/models/taskinstance.py: _defer_task
  • ./airflow/models/taskinstance.py: _handle_reschedule
  • ./airflow/models/taskinstance.py: get_task_instance
  • ./airflow/models/taskinstance.py: _clear_xcom_data
  • ./airflow/models/taskinstance.py: _set_state
  • ./airflow/models/taskinstance.py: _get_dagrun
  • ./airflow/models/taskinstance.py: _check_and_change_state_before_execution
  • ./airflow/models/taskinstance.py: _register_asset_changes_int
  • ./airflow/models/taskinstance.py: save_to_db

@rawwar
Copy link
Copy Markdown
Contributor Author

rawwar commented Nov 30, 2024

Once #44496 is merged, I need to pull changes and fix remaining failing tests

@rawwar rawwar mentioned this pull request Nov 30, 2024
90 tasks
@potiuk
Copy link
Copy Markdown
Member

potiuk commented Nov 30, 2024

Once #44496 is merged, I need to pull changes and fix remaining failing tests

Merged ! Go ahead!

@rawwar rawwar marked this pull request as ready for review December 1, 2024 03:55
@rawwar rawwar requested a review from potiuk December 1, 2024 03:55
@potiuk
Copy link
Copy Markdown
Member

potiuk commented Dec 1, 2024

Rebased and mered in #44540 .. Almost out-of the box after I rebased - and you are co-author @rawwar :)

@potiuk potiuk closed this Dec 1, 2024
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.

4 participants