jedcunningham commented on PR #43368:
URL: https://github.com/apache/airflow/pull/43368#issuecomment-2523903356
Sorry, I should have spent more time digging into this - I leaned too much
on the title/newsfrag/docs changes.
I will say, however, that the docs/newsfragment on this is misleading/wrong.
It does still work for static expansions. This also only blocks task group
expansions, expanding bare tasks with dynamic input from another task still
parses and ultimately fails when running:
```
@task
def get_input():
return ["world", "moon"]
@task(trigger_rule="always")
def hello(input):
print(f"Hello, {input}")
hello.expand(input=get_input())
```
I'd be in favor of removing this from 2.10.4 for now, then go in and block
both bare tasks and task groups, and document it more accurately too. (To be
clear, just saying we pull it based on timing, if we can get it before we merge
the 2.10.4 changes I'm completely happy with that too.)
--
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]
