Re: [PR] check transform job status before deferring SageMakerTransformOperator [airflow]

2024-01-10 Thread via GitHub
phanikumv merged PR #36680: URL: https://github.com/apache/airflow/pull/36680 -- 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...@airflo

Re: [PR] check transform job status before deferring SageMakerTransformOperator [airflow]

2024-01-09 Thread via GitHub
Lee-W commented on code in PR #36680: URL: https://github.com/apache/airflow/pull/36680#discussion_r1446847414 ## airflow/providers/amazon/aws/operators/sagemaker.py: ## @@ -700,8 +700,24 @@ def execute(self, context: Context) -> dict: raise AirflowException(f"Sagem

Re: [PR] check transform job status before deferring SageMakerTransformOperator [airflow]

2024-01-09 Thread via GitHub
dirrao commented on code in PR #36680: URL: https://github.com/apache/airflow/pull/36680#discussion_r1446340539 ## airflow/providers/amazon/aws/operators/sagemaker.py: ## @@ -700,8 +700,24 @@ def execute(self, context: Context) -> dict: raise AirflowException(f"Sage

Re: [PR] check transform job status before deferring SageMakerTransformOperator [airflow]

2024-01-09 Thread via GitHub
Lee-W commented on PR #36680: URL: https://github.com/apache/airflow/pull/36680#issuecomment-1882853941 The static check failure should be fixed in https://github.com/apache/airflow/pull/36677 -- This is an automated message from the Apache Git Service. To respond to the message, please l