GayathriSrividya commented on PR #68720: URL: https://github.com/apache/airflow/pull/68720#issuecomment-4757878320
Thanks for the review and for pointing this out clearly. After digging into it more, I agree I did not target the actual issue correctly. My changes were focused on avoiding the immediate [job_id=None](vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/code/electron-browser/workbench/workbench.html) failure, but they do not address the more important problem you described: in this case the deferrable path is not truly deferring early, because it still depends on when the launcher process yields enough information. So I’m going to close this PR rather than keep pushing an incomplete direction. If I come back to this, I’ll start from the actual deferrable execution semantics and propose the approach in the issue first before sending another PR. Thanks again for the correction. -- 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]
