Taragolis commented on PR #25980: URL: https://github.com/apache/airflow/pull/25980#issuecomment-1232875657
Sorry I've been sick for a couple of days. > I had myself convinced that we should simply remove the conn type immediately, thinking it was just a UI feature, because it would not break anything so there's no real reason to keep around for a deprecation period. Personally for me it doesn't matter remove it now or later. Might be helpful for user to know about deprecation state of Connection in the UI. >And, importantly, it is used in "generic transfer" operator. So, i think if you remove s3 hook, you'd actually break usage of s3 with generic transfer operator. So maybe we shouldn't deprecate s3 conn type at all. Can you look into this and let me know what you think? Hmmm... might be I miss something. But do we actually have this generic transfer operator? All operator which uses for transfer in/out to S3 uses aws_conn_id with "aws_default" or None as connection id. -- 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]
