potiuk commented on PR #33718: URL: https://github.com/apache/airflow/pull/33718#issuecomment-2509077574
Is this one ready to review and we think we can merge it in. Also I think it relates with our discussions on making deferrable the "default" for Airflow 3 and it is part of the discussion "Do we actually pay enough attention for deferrable timeouts and faiilure scenarios to make it "first-class" replacement for other types of sensors - so maybe we should prioritize this one - I saw a number of people commenting before, but I am not sure what the status is after so many back/forth and long conversations. .. So maybe we can somewhat restart this one and get more people who commented before and understand more in detail about the problems? cc: @eladkal Pinging those who were active here * @uranusjr * @robg-eb * @hussein-awala * @kaxil Also @thesuperzapper -> re: https://github.com/apache/airflow/issues/36090#issuecomment-2368810776 which seems very much relevant to this one, as I think we need to agree on strategy of how to treat "exceptional" cases for deferrable operators in a consistent way. I personally know too little in this area to make a meaningful (and correct) feedback, But I have a feeling this anbd #36090 would need to be addressed if we want to seriously continue discussion started here https://lists.apache.org/thread/3m7vjwcbvodnhrklo69s3j8s8pp7nm6o -- 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]
