Conversation
airflow/models/abstractoperator.py
Outdated
There was a problem hiding this comment.
Unrelated, but Mypy is complaining. context here is implemented as a MutableMapping (although we supply a type stub to say it’s not to prevent users from getting a wrong idea), so this is fine.
bbf31ef to
72d55bd
Compare
This has two parts: First, the existing extra links mechanism relies on the operator being unmapped, so we do that for most operators to keep working. Second, the UI needs to be slightly updated to ask for those links at the right time -- namely, it must only make requests against unmapped tasks, not the mapped ones.
72d55bd to
4a6632e
Compare
Hmmm. Ideally It would be nice if each individual mapped task could have it's own mapped links too. For example if your mapped task submits jobs to EMR/Databricks etc then being able to have mapped links to the remote logs/job is desirable. |
Yes, this is what this PR intends to do. What we’re currently doing is completely the other way around: we display the links against the entire mapped task (and that doesn’t work well for most things), but not for individual (mapped) task instances. This PR removes the former, and adds the links for the latter. |
Close #25360. Two parts in this: