milton0825 commented on issue #5085: [WIP][AIRFLOW-594] Add dag source plugin 
hooks
URL: https://github.com/apache/airflow/pull/5085#issuecomment-485343040
 
 
   @drewsonne 
   I think this PR and AIP-5 should share the same fetcher interface like 
`AirflowPluginDagSource` but I would probably rename it to `BaseDagFetcher` 
with two methods you proposed. AFAIK the only difference is that you propose to 
load the DAG once during startup (correct me if I am wrong) and AIP propose to 
fetch from remote periodically.  

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