fat-catTW commented on code in PR #74497: URL: https://github.com/apache/airflow/pull/74497#discussion_r4306444972
########## providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/triggers/job.py: ########## Review Comment: Thanks for the update. I think the same ordering issue remains in the non-deferrable `KubernetesJobOperator.execute()` path, which extracts XCom from `self.pods` before the suspension-aware Job polling. Could we defer that extraction until the replacement pods have completed, and add a synchronous `do_xcom_push=True` preemption regression test? -- 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]
