amoghrajesh commented on code in PR #59139: URL: https://github.com/apache/airflow/pull/59139#discussion_r2601434658
########## airflow-core/src/airflow/jobs/triggerer_job_runner.py: ########## @@ -38,6 +38,7 @@ from sqlalchemy import func, select from structlog.contextvars import bind_contextvars as bind_log_contextvars +from airflow._shared.module_loading import import_string Review Comment: Triggerer and Dagprocessor uses task sdk as of now to have the supervisor + subprocess out of box. And yes, triggerer and dag processor will be separated from airflow core. So that's more of a reason to import from task-sdk, right? Maybe I am getting confused here :), correct me if I am wrong -- 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]
