You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/06/09 21:44:43 UTC

[GitHub] [airflow] kaxil commented on pull request #16342: Remove ``[core] store_dag_code`` & use DB to get Dag Code

kaxil commented on pull request #16342:
URL: https://github.com/apache/airflow/pull/16342#issuecomment-858122998


   > Can you explain a little more why you want to delete this option?
   
   Hey yeah, was going to do it once the PR was ready for review. The thought behind it is, since this option is `True` by default in anyway and since the Webserver is relying on serialized DAGs and not file, let's remove the last thing that might use DAG files (if `store_dag_code=False`). I thought of this as many people were confusing this setting for DAG Serialization. They didn't realize that this setting is ONLY used for showing DAG Source Code in Code View (that's it), example: https://apache-airflow.slack.com/archives/CCQ7EGB1P/p1621946017296000?thread_ts=1620925239.197200&cid=CCQ7EGB1P


-- 
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:
users@infra.apache.org