The GitHub Actions job "Tests" on airflow.git has succeeded.
Run started by GitHub user ashb (triggered by ashb).

Head commit for run:
ca89d37ad2b06ae56c8c8a04a26ed803b74a3c3d / Ash Berlin-Taylor 
<[email protected]>
Create TaskMap rows when pushing XCom values from the Task Execution Interface

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 we can revisit the decision not to enforce
this later.

Report URL: https://github.com/apache/airflow/actions/runs/13076688538

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to