[GitHub] [airflow] uranusjr opened a new pull request, #24971: Move mapped kwargs introspection to separate type

[GitBox <gi...@apache.org>]

uranusjr opened a new pull request, #24971:
URL: https://github.com/apache/airflow/pull/24971

   First part toward #24489.
   
   To support list-of-dicts task-mapping (expand_kwargs), we need different logic in many places of task-mapping to handle the two difference "shapes" of data. Instead of putting conditionals everywhere, we should use polymorphism to switch between different implementations.
   
   This PR lays the foundation to this. A new class DictOfListsExpandInput is added to wrap kwargs received from expand(), and related logic is moved from MappedOperator to that class. It should be quite obvious in various places that I'm "leaving room" for something else; if you feel some abstraction is unnecessary (it's probably not), check #24838 to make sure.
   
   I also included some trivial annotation and code formatting stuffs here since they don't warrant a separate PR and fit as well here as anywhere else.


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

[GitHub] [airflow] uranusjr merged pull request #24971: Move mapped kwargs introspection to separate type

[GitBox <gi...@apache.org>]

uranusjr merged PR #24971:
URL: https://github.com/apache/airflow/pull/24971


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