In PR https://github.com/apache/airflow/pull/63452, I remove old custom
Airflow OTEL "wrapper" interfaces.

*The important question is whether we need to keep any backcompat layer.*

The conclusion I've come to, after thinking it through with Jarek and
Christos, is that these interfaces were not really intended to be
user-facing, and the likelihood that users put them in their dags is
sufficiently low that we can simply remove them.

One thing to point out is that these classes were never released *in their
present locations.*

They were initially moved to their present locations in
https://github.com/apache/airflow/pull/56187, notably without maintaining
any backcompat importability from the old location.  It's notable because,
given that there were many reviewers of that PR, it's suggestive of the
conclusion that people did not think of this as user-facing code.

Additionally, there is no documentation that suggests that users use these
interfaces.

These interfaces were made unnecessary by
https://github.com/apache/airflow/pull/62554 and some other PRs that
preceded it.  So I think the appropriate course of action is to remove them.

But if anyone has any concerns, we can convert this to a discussion item.

Thanks for your consideration.

Daniel

Reply via email to