olivermeyer commented on issue #16299: URL: https://github.com/apache/airflow/issues/16299#issuecomment-856707637
@eladkal I'm happy to take a shot at opening a PR for this, but before I do I'd like to validate my idea. Removing the uniqueness check entirely might cause issues for users who rely on it (whether they know that they rely on it or not), so the better solution, even though it bloats the code, is probably to add a parameter to the operator to indicate whether it should perform this check or not. I would imagine it to be simply something like a `check_job_name_uniqueness` parameter (or something more concise), which essentially skips [this](https://github.com/apache/airflow/blob/db63de626f53c9e0242f0752bb996d0e32ebf6ea/airflow/providers/amazon/aws/operators/sagemaker_training.py#L95-L107) whole block if set to true. Does that make sense to you? -- 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. For queries about this service, please contact Infrastructure at: [email protected]
