Kache commented on PR #35614: URL: https://github.com/apache/airflow/pull/35614#issuecomment-1810738948
Thanks for elaborating on the broader implications. Isn't "trying to move out FAB to separate provider" a separate feature/refactor concern? It's simply currently the case that `User` _is_ referenced in `DagRunNote` as `"ab_user.id"`, and this change is just making the current state of things explicit enough for Python to load modules in dependency order. This change shouldn't make moving out FAB more difficult either. Would you guys think it sufficient to add `TODO` comments in reference to #32210 or additional notes elsewhere? Alternatively, perhaps the foreign key constraint could be removed entirely, but that seems to require a migration which requires a bit more ceremony that I'm not familiar with. -- 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]
