SameerMesiah97 commented on PR #59691: URL: https://github.com/apache/airflow/pull/59691#issuecomment-4670762251
> @SameerMesiah97 have you tested this fix locally with Dynamic Task Expansion, any logs to prove that it the change fixes the issue? Maybe a stupid question, but that could help approve the PR as for me code is looking good, but of course I haven't tested it so I cannot confirm this will actually fix the issue. Nice work BTW. If by Dynamic Task Expansion you meant Dynamic Task Mapping, then yes. I have done several tests using a variety of DAG shapes at mutliple points in time whilst working on this PR. If you scroll up, you will see me reporting the results after each test run. I will do another test as soon as I am able to and I will provide all the details as well. -- 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]
