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

Head commit for run:
3a168154afb400f6e616b35c9f68574b47d72e48 / Tzu-ping Chung <[email protected]>
Remove is_mapped attribute

After task group mapping is implemented, the semantic of an operator
being "mapped" becomes ambiguous, since it can mean either "the operator
is itself mapped" (i.e. a MappedOperator) or "the operator is mapped to
multiple task instances" (can be either a MappedOperator, or inside a
MappedTaskGroup). This makes the use of the "is_mapped" flag awkward and
induces some subtle bugs found during testing.

Therefore, this PR proposes to remove the is_mapped flag, and rely
entirely on isinstance checks to more explicitly identify the two kinds
of task mapping constructs instead. A couple of bugs are also caught
during refactoring.

Note that the is_mapped fields in *serialized formats* (serialized
operator, and JSON responses returned by API calls) are not removed, so
front end code is not affected. The fields are simply modified to also
rely on isinstance checks.

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

With regards,
GitHub Actions via GitBox


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

Reply via email to