kaxil commented on PR #67356:
URL: https://github.com/apache/airflow/pull/67356#issuecomment-5942939206

   Closing this. Overriding a `DictOfListsExpandInput` argument with 
`[new_param] * (args.map_index + 1)` changes the length of that one list, and 
with it the cross product of all expanded arguments, so the other arguments 
resolve to the wrong values: with `expand(a=[1, 2, 3], b=[10, 20, 30])`, 
`--map-index 4` and `--task-params '{"b": 99}'`, the task gets `a=1` instead of 
`a=2`, as reported in review in July. The tests only cover single-argument 
expansion. It also makes `--task-params`, which sends params to the task, 
override expand arguments.


-- 
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]

Reply via email to