uranusjr opened a new pull request #22272: URL: https://github.com/apache/airflow/pull/22272
This makes multiple ti-related views take an additional parameter `map_index` (optional, defaults to -1 if not given for unmapped tasks). But I’m not really sure how to actually _pass_ the argument in those. Also `map_index` is added to `list_columns` for views to models with that field. Some oddities not handled in this PR that need to be fixed later: * `/success` and `/failed` set state to a task (and potentially its upstreams/downstreams). It feels a bit weird to be able to set the state to only one mapped task? Also upstream/downstream makes less sense if any of those are mapped. So I’m currently leaving these views alone (i.e. setting success/failed to a mapped task sets the state of all mapped tis). We can perhaps add some UI components to optionally do more fine-grained state modification. * `SlamissModelView` is currently unchanged awaiting #22184). * `LogModelView` is currently unchanged. We need to add `map_index` to the `Log` model first. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
