dstandish commented on code in PR #25249: URL: https://github.com/apache/airflow/pull/25249#discussion_r928861034
########## tests/api_connexion/endpoints/test_task_endpoint.py: ########## @@ -332,6 +336,7 @@ def test_get_tasks_mapped(self): "execution_timeout": None, "extra_links": [], "is_mapped": True, + # todo: should be "outlet_datasets": [], but does not work for mapped Review Comment: fundamentally, it doesn't make sense to not support datasets just because a task is mapped. that sorta means mapped tasks aren't first class citizens. with mapped tasks, we'll soon be able to create "map kwargs" essentially (so iterating over list(dict) instead of list(primitive), which i think should unlock mapping the outlets param too? i dunno. it needs thought. but, just doesn't seem right for it to not be supported, for a datasets API -- are mapped tasks not supposed to also be used for data? -- 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