uranusjr commented on pull request #22334: URL: https://github.com/apache/airflow/pull/22334#issuecomment-1071199592
Alright hopefully this fixes everything. Mostly changing the test code, but I also needed to change how `airflow task test` works conceptually to make this foreign key work (see the commit referencing this command). Currently `airflow task test` would create a “fake” DAG run in-memory to run just that task, but this does not work well with database-level foreign key. So instead I make this command always create a “real” DAG run in the database, and delete it (with everything the run generates because `ON DELETE CASCADE`) after the task is run. Personally I think this is better because now this command is no longer a weird outlier on database integrity, but not sure if this is an acceptable semantic change. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
