aoen commented on issue #4396: [AIRFLOW-3585] - Add edges to database
URL: https://github.com/apache/airflow/pull/4396#issuecomment-469942061
 
 
   Sorry Ash, missed your reply. Kevin is correct, we should extend SimpleDag 
to contain edge information and store the serialized representation of this in 
the DB. My feeling is we should also store entire serialized representations of 
DAGs (e.g. JSON5 or have some discussion to agree on format) rather than as 
columns in the DB, both for performance reasons to avoid n+1 queries when we 
use the SimpleDAG representations for the Scheduler and Webserver, unless there 
is a good reason not to do this.

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


With regards,
Apache Git Services

Reply via email to