[GitHub] [airflow] uranusjr commented on pull request #22904: Call mapped_dependants only on the original task

2022-04-11 Thread GitBox


uranusjr commented on PR #22904:
URL: https://github.com/apache/airflow/pull/22904#issuecomment-1095455487

   H… does the CI run long-running tests by default…?


-- 
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-unsubscr...@airflow.apache.org

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



[GitHub] [airflow] uranusjr commented on pull request #22904: Call mapped_dependants only on the original task

2022-04-11 Thread GitBox


uranusjr commented on PR #22904:
URL: https://github.com/apache/airflow/pull/22904#issuecomment-1095447937

   Yeah I just added those two to the test DAG (_without the proposed fix) and 
CI still passes.


-- 
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-unsubscr...@airflow.apache.org

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



[GitHub] [airflow] uranusjr commented on pull request #22904: Call mapped_dependants only on the original task

2022-04-11 Thread GitBox


uranusjr commented on PR #22904:
URL: https://github.com/apache/airflow/pull/22904#issuecomment-1095358722

   Hmm that looks weird. The bug is when an upstream task finishes and wants to 
scan its downstream tasks; so whether the upstream is mapped from a literal or 
not should not matter.


-- 
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-unsubscr...@airflow.apache.org

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