raphaelauv commented on PR #41463:
URL: https://github.com/apache/airflow/pull/41463#issuecomment-2308907278

   ```
    It was still changing behaviour for 2.10 .. because we did not want to 
suddenly change behaviour of those sensors
   ```
   yes previous PR was fixing a bug introduce in airflow 2.7.1 that suddenly 
changed the behaviour of sensors
   
   ```
   The problem with the change is it is implemented now is that you can't 
import "AirflowPokeFailException" And "FailPolicy" from "airflow 2" in 
providers - because it is just missing in Airflow 2.8 - 2.10 and providers 
simply will not work for 2.8. 2.9. 2.10. And since we do not have Airlfow 3 for 
quite some time (6 months) - it means they won't work at all. And having it in 
2.10 would not help to much (because the providers will still have to work for 
2.8-2.9 where it is missing).
   ```
   
   my previous PR was compatible with 2.9 and 2.10, all tests were green  -> 
   
![image](https://github.com/user-attachments/assets/6848836c-20f4-4c42-a2e2-dd54a0887b7b)
   
   ---
    ```
   ...  but with "task.sdk" ...
    ```
   
   thanks for the info I will have a look to the AIP
   
https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-72+Task+Execution+Interface+aka+Task+SDK
   


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