dilnazanlid commented on PR #72369: URL: https://github.com/apache/airflow/pull/72369#issuecomment-5586471218
> Since this PR is called “Move” in the title, it should also rewire the internals to actually use the newly added importers, and delete the no-longer-needed code after rewiring. Otherwise, this probably should be rebranded to _preparing_ for the move (which would still be fine since the added code is unit-tested). I would rather have this PR separately - because the actually replacing the `dag_processing/importers` with `sdk/importers` also includes moving away from filesystem path to DagDefinition, and it is quite big change too. So let me rename the PR to make it clear that this is just preparation. Thanks! -- 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]
