You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/02/17 07:41:23 UTC

[GitHub] [airflow] uranusjr opened a new pull request #21635: Fixup how we expand task

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


   This rewrites `expand_mapped_task()` to inspect `mapped_kwargs` and make itself useful for mapping against literals (which are not recorded in `TaskMap`), and should work for things like `map(a=upstream_a(), b=upstream_b())` (assuming it is correctly called when both upstreams finish—this is not the case now, we need to fix that later).
   
   I think the logic is still not quite right (e.g. mapping against a *mapped* upstream), but this new logic should be useful for what we're going to need to fix.


-- 
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 closed pull request #21635: Fixup how we expand task

Posted by GitBox <gi...@apache.org>.
uranusjr closed pull request #21635:
URL: https://github.com/apache/airflow/pull/21635


   


-- 
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 commented on pull request #21635: Fixup how we expand task

Posted by GitBox <gi...@apache.org>.
uranusjr commented on pull request #21635:
URL: https://github.com/apache/airflow/pull/21635#issuecomment-1042743311


   Damn, it’s difficult to split this out of my local edits. Let’s try submitting all the things in one PR and see if that’s workable.


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