KevinYang21 commented on issue #5085: [WIP][AIRFLOW-594] Add dag source plugin 
hooks
URL: https://github.com/apache/airflow/pull/5085#issuecomment-484707092
 
 
   Personally I would prefer to put such logic inside core Airflow instead of 
as a plugin. This doesn't look like a framework that AIP-5 can be implemented 
on top of. I prefer the way described in AIP-5 that we switch to use the 
DagFetcher abstraction, which is an independent core Airflow component, right 
before we parse the DAG or periodically fetch the DAG in an independent process.

----------------------------------------------------------------
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

Reply via email to