[GitHub] [airflow] ashb commented on a change in pull request #17719: Change TaskInstance and TaskReschedule from execution_date to run_id

[GitBox <gi...@apache.org>]

ashb commented on a change in pull request #17719:
URL: https://github.com/apache/airflow/pull/17719#discussion_r701408236



##########
File path: UPDATING.md
##########
@@ -228,6 +228,14 @@ Now that the DAG parser syncs DAG permissions there is no longer a need for manu
 
 In addition, the `/refresh` and `/refresh_all` webserver endpoints have also been removed.
 
+### TaskInstances now *require* a DagRun
+
+Under normal operation every TaskInstance row in the database would have DagRun row too, but it was possible to manually delete the DagRun and Airflow would still execute the TaskInstances.
+
+In Airflow 2.2 we have changed this and now there is a database-level foreign key constraint ensuring that every TaskInstance has a DagRun row.
+
+Before updating to this 2.2 release you will have to manually resolve any inconsistencies (add back DagRun rows, or delete TaskInstances) if you have any "dangling" TaskInstance" rows.
+

Review comment:
       Probably should yes (though having "extra" config items doesn't cause any harm)




-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org