[GitHub] [airflow] dstandish commented on issue #5263: [AIRFLOW-4490] dag_run.conf should be an empty dictionary by default instead of None

[GitBox <gi...@apache.org>]

dstandish commented on issue #5263: [AIRFLOW-4490] dag_run.conf should be an empty dictionary by default instead of None
URL: https://github.com/apache/airflow/pull/5263#issuecomment-491671014
 
 
   > Is there ever a case where the DagRun is not committed to that database immediately?
   Intuitively, it seems unlikely in actual execution.  But if you are experimenting in interactive console, as you have demonstrated, it is certainly possible.
   Another alternative approach to this would be to leave default value alone, but just add a `get_conf` method, just like there is now with `get_dag`.  `get_conf` could return an empty dict if there is no conf.
   

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


With regards,
Apache Git Services