dstandish opened a new pull request, #63452: URL: https://github.com/apache/airflow/pull/63452
The removals in this PR were actually never in an Airflow release. This code was added or moved to the current location in PR https://github.com/apache/airflow/pull/56187. Prior to that PR, the code was imported from the `airflow.traces` package. All of this code is made unnecessary by https://github.com/apache/airflow/pull/62554 and some PRs that preceded it. The important question for this PR is, do we need to keep these interfaces around for backward compatibility purposes. But I think we don't. I think we can just remove it. And the reason why, is none of these interfaces appeared in the documentation. Additionally, no one identified backcompat as an issue in https://github.com/apache/airflow/pull/56187, the PR that moved them to their present location (and did not maintain a backcompat layer); I think this also is indicative of the conclusion that people don't think of this as user-facing. So my best guess is, these interfaces weren't really intended to be used by users. Additionally, I think the likelihood that users actually used these classes is quite low. So I think the best course of action is to just remove them. -- 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]
