You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "ashb (via GitHub)" <gi...@apache.org> on 2023/08/30 11:09:08 UTC

[GitHub] [airflow] ashb commented on pull request #33746: Refactor: remove unused state - SHUTDOWN

ashb commented on PR #33746:
URL: https://github.com/apache/airflow/pull/33746#issuecomment-1698956131

   We need to add `TaskInstanceState.SHUTDOWN` back even if nothing uses it -- this counts as a breaking change since it has been documented and exists in the code.
   
   By removing this as we have done this caused import errors. One such use I know of is https://github.com/astronomer/astro-sdk/blob/75838143c1b3181b636259e5c39fa27a70d89c3c/python-sdk/src/astro/sql/operators/cleanup.py#L167
   
   (This showed up as a DAG parse error on our tests against Airflow main)


-- 
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