[GitHub] [airflow] vandonr-amz commented on pull request #29245: fix code checking job names in sagemaker

2023-02-21 Thread via GitHub
vandonr-amz commented on PR #29245: URL: https://github.com/apache/airflow/pull/29245#issuecomment-1439042642 hello @dimberman we just had a new system test failure because of this issue (getting throttled on too many listing requests). Any chance you can re-assess your review ? It'd be

[GitHub] [airflow] vandonr-amz commented on pull request #29245: fix code checking job names in sagemaker

2023-02-02 Thread via GitHub
vandonr-amz commented on PR #29245: URL: https://github.com/apache/airflow/pull/29245#issuecomment-1414503882 sorry for the force-push, there was a clandestine commit that made its way into this PR :/ -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] vandonr-amz commented on pull request #29245: fix code checking job names in sagemaker

2023-01-31 Thread via GitHub
vandonr-amz commented on PR #29245: URL: https://github.com/apache/airflow/pull/29245#issuecomment-1410954197 > We're only counting jobs whose name matches [here](https://github.com/apache/airflow/blob/main/airflow/providers/amazon/aws/hooks/sagemaker.py#L1022) Oh yes my bad, I read

[GitHub] [airflow] vandonr-amz commented on pull request #29245: fix code checking job names in sagemaker

2023-01-30 Thread via GitHub
vandonr-amz commented on PR #29245: URL: https://github.com/apache/airflow/pull/29245#issuecomment-1409097093 need to fix transform tests, I forgot that when pushing :| -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use