Re: [PR] Enable mapped task argument overrides in `airflow tasks test` [airflow]

["kaxil (via GitHub)" <gi...@apache.org>]

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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org