potiuk commented on issue #35644: URL: https://github.com/apache/airflow/issues/35644#issuecomment-1816325187
> The problem a bit more deep, all current exceptions is a part of [Public Interface of Airflow](https://airflow.apache.org/docs/apache-airflow/stable/_api/airflow/exceptions/index.html) which I think it should not be, there are only couple exceptions which should be a part of public interface and IMO `AirflowException` and `AirflowTaskTimeout ` should not be a part of public interface but we have what we have. Yet another case when we should decide why it not breaking changes. Yes. I think we should remove those "non -public" exceptions from the list. The list **might** contain things that should not be there and we are free to remove them if they found its way by mistake. Might be a good idea to do so as a separate PR. But even if not - I think changing the base for exception does not qualify as "Breaking change" IMHO. It's still an exception and it does not change the usage of it. -- 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]
