drewsonne edited a comment on issue #5085: [WIP][AIRFLOW-594] Add dag source plugin hooks URL: https://github.com/apache/airflow/pull/5085#issuecomment-485102939 @KevinYang21 The intention here is to build some hooks that can facilitate loading DAGs from anywhere. When I say building as a plugin, there's nothing contrary to AIP-5. AIP-5 could be in the core, and be built on top of this. The intention would be that any DAG loading method used in the core would use the same hooks as the plugins. The philosophy here is to use the same code to extend core functionality as used to extend plugin functionality to provide everyone with good solid hooks for performing extensions to Airflow without having to do PR's on the core. But just to confirm, there is nothing contradictory with AIP-5 and this PR. AIP-5 could use the hooks in this PR and still be distributed with the core airflow. Side note: Using plugin hooks, means you can have pieces of code which exist in seperate repositories and users can install at their will and if there is a desire within the core community, those repositories can be brought into airflow core through normal PR process. But it reduces the need for actual PR's into the core on whether code is suitable to go in the airflow core.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
