ashb opened a new pull request, #46319:
URL: https://github.com/apache/airflow/pull/46319

   This is needed so that we can have the scheduler create the expand TIs for 
the downstream tasks.
   
   Note, that we don't enforce that this is set when it needs to be on the 
server side, as the only way for us to know if we need to or not (and 
conversely, when it's set when it _doesn't_ need to, which is no effect other 
than creating a small DB row that nothing will ever read) as doing that would 
involve loading the serialized DAG to walk the structure which is a relatively 
expensive operation.
   
   We could improve that by "pre-computing" some of the info on what tasks are 
actually mapped or not before serialization so we wouldn't have to walk the 
task groups to find out, but that wouldn't do anything about the need to load 
the serialized DAG which is the expensive part.
   
   If this turns out to be a problem to not be enforcing this  is passed when 
required we can revisit the decision later.
   
   Relates to #44360


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

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to