atamagrawal commented on PR #72444:
URL: https://github.com/apache/airflow/pull/72444#issuecomment-5515828911

   @jroachgolf84 
   
   The CI failure was in the Airflow 3.0.6 compatibility job, not in the normal 
provider tests.
   **What broke:** On Airflow 3.0.6, TaskInstanceState is not available from 
airflow.sdk. Our first change used:
   
   from airflow.sdk import TaskInstanceState
   That works on newer Airflow, but on 3.0.6 it fails at import time.
   
   Fix: Import it through the compat layer already used in this file:
   
   from airflow.providers.common.compat.sdk import AirflowException, 
TaskInstanceState, conf
   On newer Airflow it uses airflow.sdk. On 3.0.6 it falls back to 
airflow.utils.state. Same behavior, compatibility tests pass.


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

Reply via email to