[GitHub] [airflow] o-nikolas commented on a diff in pull request #29245: fix code checking job names in sagemaker

2023-02-27 Thread via GitHub
o-nikolas commented on code in PR #29245: URL: https://github.com/apache/airflow/pull/29245#discussion_r1119184791 ## airflow/providers/amazon/aws/operators/sagemaker.py: ## @@ -106,6 +108,41 @@ def _create_integer_fields(self) -> None: """ self.integer_fields

[GitHub] [airflow] o-nikolas commented on a diff in pull request #29245: fix code checking job names in sagemaker

2023-02-14 Thread via GitHub
o-nikolas commented on code in PR #29245: URL: https://github.com/apache/airflow/pull/29245#discussion_r1106169712 ## airflow/providers/amazon/aws/operators/sagemaker.py: ## @@ -106,6 +108,41 @@ def _create_integer_fields(self) -> None: """ self.integer_fields

[GitHub] [airflow] o-nikolas commented on a diff in pull request #29245: fix code checking job names in sagemaker

2023-01-31 Thread via GitHub
o-nikolas commented on code in PR #29245: URL: https://github.com/apache/airflow/pull/29245#discussion_r1092353835 ## airflow/providers/amazon/aws/operators/sagemaker.py: ## @@ -679,11 +697,11 @@ def __init__( self.check_interval = check_interval self.max_inges